Skip to content

Making a Modal that enables user to share a course with their friends (UI Logic)#20

Open
Mau-MD wants to merge 119 commits intomainfrom
clone_course
Open

Making a Modal that enables user to share a course with their friends (UI Logic)#20
Mau-MD wants to merge 119 commits intomainfrom
clone_course

Conversation

@Mau-MD
Copy link
Owner

@Mau-MD Mau-MD commented Jul 20, 2022

App Context

Learning U is a platform where software engineers can go and learn new technologies in an organized way. They can track their progress and get suggestions about what courses to take in both visual and written ways.

PR Context

Every time a course is created using the form, it generates fresh resources using an algorithm, so even if a user tries to generate a course with the same day twice, it can get different content. There's no way to share or keep a course so it preserves the same content.

This PR

Adds a UI Modal where the user gets a code that it can share with its friends as well as an extra form input inside the course create to input that course code.
CleanShot 2022-07-19 at 17 35 03@2x
CleanShot 2022-07-19 at 17 34 53@2x

Next

Add the Backend Logic to be able to clone the course.

IMPORTANT:

Since this PR was based in another one, a lot of commits appear here even if they are not related to this PR. In reaility commits start here: Adding chakra ui modal component

How to preview a PR on Vercel

  • Go to /login and log in with username admin and password admin123
  • Go to /dashboard

@vercel
Copy link

vercel bot commented Jul 20, 2022

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

Name Status Preview Updated
learning-u ✅ Ready (Inspect) Visit Preview Jul 20, 2022 at 0:36AM (UTC)

@Mau-MD Mau-MD requested review from ImaMonsta and nosoyilse July 20, 2022 00:36
@Mau-MD Mau-MD added the waiting for another pr Another PR needs to be approved before this gets reviewed label Jul 20, 2022
@nosoyilse
Copy link
Collaborator

Very good PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

waiting for another pr Another PR needs to be approved before this gets reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants