Feat/lesson info modal

[recepcankaya]

Change SideDrawer component to shadcn Dialog component.

API Updates

• Add shadcn dialog component

New Features (required)

• to test the new component, please open tracks/intro-to-ethereum/1

Fixes #67
Will close issue #67 on merge

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
academy-turbo	✅ Ready (Inspect)	Visit Preview	Jan 19, 2024 0:46am
academy-turbo-storybook	✅ Ready (Inspect)	Visit Preview	Jan 19, 2024 0:46am

[wolovim]

thanks for jumping in!

• first, looks like you need to rebase from main and resolve conflicts
• based on the image below, it seems that question titles arent displayed
• no styles are applied on selection of answer (or hover)
• question submit button color doesnt match figma
• form cancel button text and color are the same (unreadable) until you hover

[recepcankaya]

@wolovim i have seen a new component called LessonInformationalModal and mine is LessonInfoModal. I am confused rn. Are they supposed to do the same thing?

[wolovim]

damn. i think its the same @recepcankaya. @Markkos89 mentioned the QuestionsModal in this comment in #70, but the InformationalModal is created in the same PR:

created new LessonQuestionsModal (cc @recepcankaya sorry for creating it, mate, I hope my code helps in the worst case)

github email notifications are noisy and can be easy to miss, so next time theres potential overlap like this, lets make sure a DM goes out @Markkos89.

looks like theres still at least a bit of styling to do from markko's component - the selected answer is unstyled, for example. to wrap up this unit of work, i suspect its quickest to abandon this component and finish the last of any styling from the LessonInformationalModal? sorry to see work get thrown out; lets tighten up communication to avoid that in the future.

[wolovim]

again, sorry to see this work rendered obsolete @recepcankaya. going to close this PR and encourage any new iterations start fresh from the latest on the main branch.