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

spotify mockup #106

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

spotify mockup #106

wants to merge 5 commits into from

Conversation

smily342
Copy link

Netlify link

Add your Netlify link here.
PS. Don't forget to add it in your readme as well.

Collaborators

Add any collaborators here, as well as in the title of the PR.
https://github.com/Sherrydev11
https://github.com/Heleneabrahamsson

@smily342
Copy link
Author

after mutiples correction you will find our website here : https://shimmering-raindrop-59fa5c.netlify.app/

@JennieDalgren JennieDalgren self-assigned this Oct 17, 2024
Copy link

@JennieDalgren JennieDalgren left a comment

Choose a reason for hiding this comment

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

Great job with the project!
It meets all requirements.

Props are being passed correctly from App.jsx to Album (). The album data is passed as a prop, which is excellent for reusability and scalability.

It seems like you have pushes a lot of "extra files" to github. Make sure to always have the .gitignore file in your repo and check the files before you push your changes to github.

Keep up the good workk. It seems like you work great together 🚀

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.

3 participants