Skip to content
This repository was archived by the owner on Apr 21, 2024. It is now read-only.

Conversation

@Fomonyuytar-Joseph
Copy link
Contributor

For an example of how to fill this template out, see this Pull Request in the Wiki.

Description

Add Skills to components folder in the src
add form to Skills.js

Acceptance Criteria

Type of Changes

Type
🐛 Bug fix
✨ New feature
🔨 Refactoring
💯 Add tests
🔗 Update dependencies
📜 Docs

Updates

After

Skills

@netlify
Copy link

netlify bot commented Oct 12, 2022

Deploy Preview for dev-cv-ycc ready!

Name Link
🔨 Latest commit e131244
🔍 Latest deploy log https://app.netlify.com/sites/dev-cv-ycc/deploys/6346e0e676a65c00087a6bf4
😎 Deploy Preview https://deploy-preview-29--dev-cv-ycc.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@Fomonyuytar-Joseph
Copy link
Contributor Author

Hi @YuriDevAT ,thanks for your response sorry I got confused that why I kept closing the PR😅 I thought it was the right thing to do

Copy link
Member

@YuriDevAT YuriDevAT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good now 🙂 . One little suggestion, the Skills component should appear before the blank CV component, to be together with the other forms. I will merge after your change.

@Fomonyuytar-Joseph
Copy link
Contributor Author

ok I have done so as required

@YuriDevAT
Copy link
Member

Great work @Fomonyuytar-Joseph. 🥳

@YuriDevAT YuriDevAT merged commit 1fa8b54 into YurisCodingClub:main Oct 12, 2022
@YuriDevAT YuriDevAT added the hacktoberfest-accepted Hacktoberfest contribution label Oct 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

hacktoberfest-accepted Hacktoberfest contribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants