-
Notifications
You must be signed in to change notification settings - Fork 5
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
Web 6 contributors section update #31
Conversation
✅ Deploy Preview for samoa-stack-overflow ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Uso can you pull latest changes from main please? and include your component with the rest in page.svelte |
But overall looks good! Might need you to work on different colors for dark and light mode for the cards. Looks good on mobile, might need to reduce space between cards. Good stuff! |
c6c685d
to
63c727b
Compare
Also don't forget to import and showcase your changes in the |
Also don't forget to reference the issue number in your PR description. |
c0b9c1e
to
b99355e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one small change and should be good to go!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one small change and should be good to go!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Malo lava!
Describe your changes
updating branch with all changes
Issue ticket number and link
Issue #19
Checklist before requesting a review