BobaTalks / pick-a-major

0 stars 4 forks source link

add button, change data, and fix icon for quiz #120

Closed Raymond0T closed 1 year ago

Raymond0T commented 1 year ago

Description

What is the purpose of your change?

-add button, change data, and fix icon for quiz (fixes #121)

What areas of the application does this change impact? (changes API, folder structure, etc)

-data file, images folder, and quiz component

What features/fixes are deferred/not handled in this PR?

-

Screenshot/link:

https://deploy-preview-120--pick-a-major.netlify.app/quiz/classes image

Checklist

netlify[bot] commented 1 year ago

Deploy Preview for pick-a-major ready!

Name Link
Latest commit 1a032851e40ff7ac397d241cffa6f5d8160c6e27
Latest deploy log https://app.netlify.com/sites/pick-a-major/deploys/64e3de6149279f00077ef4b1
Deploy Preview https://deploy-preview-120--pick-a-major.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.

Raymond0T commented 1 year ago

Is there a ticket this is connected to? Can we reference it in the description? we didn't have a ticket for it before but it is referenced now

jwu910 commented 1 year ago

image

also per the designers

Raymond0T commented 1 year ago

image

also per the designers

This would be done in another PR when we fix maximum card selected issue and also saving user's selection.

jwu910 commented 1 year ago

This would be done in another PR when we fix maximum card selected issue and also saving user's selection.

Sure that's fine. Was there a ticket already for that too?