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

Cones & Stones #37

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

Cones & Stones #37

wants to merge 266 commits into from

Conversation

JohannaBN
Copy link

@JohannaBN JohannaBN commented Jun 13, 2024

Netlify links
Frontend: https://cones-and-stones.netlify.app/
Backend: https://cones-and-stones-ppnudpghiq-lz.a.run.app/

Collaborators
[ericamechler, FridaMari]

ericamechler and others added 30 commits May 30, 2024 11:51
added routes for /, /products, /ptofuvyd/category/:category and /prod…
Moved middleware to check if bug with service unawailable is fixed.
fixed the routes for the navlinks- they are now hard-coded based on t…
Import logo and icons to assets folder and add them to the header
…performance etc. Created image-component that I then reused for both the icons in the header and also in the hero-image. Fixed so that the link to all products work and styled the hero a little
moved assets to public-folder, because of common practice for better …
Apply some styling to header
Made test to see if API-error is fixed. Removed middleware.
added button-component and implemented in hero component, also added …
Import fonts to index.html, define font styling and add css variables
JohannaBN and others added 29 commits June 13, 2024 10:41
Fixed bug with text not center on very small screens on size buttons.
Remove max-width from toggle component and place it on the footer ins…
Second try to update colors
removed classname hero-h4 which was not used
removed category/:id because we don't use it
Copy link

@AntonellaMorittu AntonellaMorittu left a comment

Choose a reason for hiding this comment

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

Dear Erica, Frida and Johanna, stort grattis! 🎓
Really impressive work, there’s so much talent here for real. I can’t believe you delivered this product developed in only 3 weeks, or rather I do as I know what you’re capable of :)

In this final project, we asked you to plan your bigger project work, further deepen your knowledge in web development and finally explore new technologies! Let me tell you, you surely succeeded in all these aspects!

I would like to summarise in some points what I think about your app:

  • Extremely well-structured and formatted code
  • Well-planned wireframe and design on Figma
  • Impressively polished final app, with smooth flow and great UX (info texts, animations, easy to navigate)
  • Flawless stripe integration and checkout flow
  • Beautiful clean UI
  • Robust and concise database data structure
  • Clever use of AI for images
  • Nailed accessibility
  • Nailed responsiveness
  • Well documented project in your README

Last but not least you did an amazing presentation during demos, proving you’re ready to be graduated developers ⭐

My congratulations and I wish you all the best in your future careers 🍀 🥇 🤝

Choose a reason for hiding this comment

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

very good you added such file! 👏

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.

4 participants