Skip to content

Project 1 Feedback#1

Open
r-pears wants to merge 2 commits intoffups:mainfrom
r-pears:feedback
Open

Project 1 Feedback#1
r-pears wants to merge 2 commits intoffups:mainfrom
r-pears:feedback

Conversation

@r-pears
Copy link

@r-pears r-pears commented Feb 15, 2025

Nice basic functionality.

However, you need to fix:

  • Use the API provided, instead of a local file.
  • Handle that logic.

Additional feedback on this code when provided.

Keep in mind for the future, but no need to fix now:

  • Alerts to the user isn't the best user experience.
  • Being able to move back and forward and changing your answers isn't the best UI.

[No pass]

@netlify
Copy link

netlify bot commented Feb 15, 2025

Deploy Preview for quizgem ready!

Name Link
🔨 Latest commit bf2a9e6
🔍 Latest deploy log https://app.netlify.com/sites/quizgem/deploys/67b060403c23200008b655e4
😎 Deploy Preview https://deploy-preview-1--quizgem.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 configuration.

@ffups
Copy link
Owner

ffups commented Mar 12, 2025

hey @r-pears I have fixed the error in my latest main push. I hope that resolves everything!

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.

2 participants