Skip to content

Displaying the course and resources inside the React App #14

Open
Mau-MD wants to merge 77 commits intomainfrom
get_current
Open

Displaying the course and resources inside the React App #14
Mau-MD wants to merge 77 commits intomainfrom
get_current

Conversation

@Mau-MD
Copy link
Owner

@Mau-MD Mau-MD commented Jul 19, 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

Right now you can create a course using the UI (New Course Form) but it's only creating it in the backend. Right now the user has no visual feedback of the course it created

This PR

This PR displays all the courses the user in enrolled in as well as all the resource that that course contains (youtube tutorials). The progress tracking doesn't work yet.

CleanShot.2022-07-19.at.16.44.53.mp4

Next

Make the progress tracking work and make the users able to complete a 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: Showing cards in react

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 19, 2022

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

Name Status Preview Updated
learning-u ✅ Ready (Inspect) Visit Preview Jul 19, 2022 at 11:50PM (UTC)

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

@nosoyilse nosoyilse left a comment

Choose a reason for hiding this comment

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

Code lgtm!

Commit size has improved a lot.

I will still make smaller PRs

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