Skip to content

Displaying course progress inside the Course Hub Page#18

Open
Mau-MD wants to merge 107 commits intomainfrom
course_progress
Open

Displaying course progress inside the Course Hub Page#18
Mau-MD wants to merge 107 commits intomainfrom
course_progress

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

Right now you can see the resources and video tutorials from one specific course, but the course progress only shows dummy data and it doesn't change even if you finish a video.

This PR

Adds the functionality to mark a video as In progress if you click on it, and to complete it if you click the Mark as completed? button. Also it shows the progress in a circular progress bar.

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

Next

Show the progress in the difficulty selection page and course card. As it only shows in the hub page.

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 resource status type

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:22AM (UTC)

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

for (const resource of resources) {
if (resource.status === "completed") totalCompleted++;
if (resource.status === "in progress") totalInProgress++;
Copy link
Collaborator

Choose a reason for hiding this comment

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

this status should be enums!

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