Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Skills Section #9

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

smolsuryansh
Copy link

Hi, I have added the skills section as you mentioned in the issues. DON'T MERGE this to MAIN branch as I couldn't test it properly (probably because of CORS configuration).

Also I apologize for ruining the layout. I couldn't think of any other way of adding the Skills section. (:sob:)

Either way, your project is a really cool project! Let me know if my code was any useful and if I can be of any more use. 😄

Some (probably) important points:

  • You can find the whole list of skills with their logos in the "/constant/skills.ts" directory. I couldn't think of any more skills so those are all I could add. To add more simply add more skills like: { name: "Example", logo: "/logos/example.png" },.

  • Test and ensure that the download and the "link" feature works with the skills section. In the Bento1.tsx take a look at line 121 and 156 to find the code added for handling download and "link" feature.

  • The changes were also made in Home.tsx.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant