Dev - now with dynamic progress bar !

sofiiagran commented 10 months ago

Description

This is a small PR where the progress bar now is dynamic

Changes

Added a function to calculate how much of a course the user have completed, called "checkProgressCourse" and "checkProgressSection"

Checklist

[x] Code has been tested locally and passes all relevant tests.
[x] Documentation has been updated to reflect the changes, if applicable.
[x] Code follows the established coding style and guidelines of the project.
[x] All new and existing tests related to the changes have passed.
[x] Any necessary dependencies or new packages have been properly documented.
[x] Pull request title and description are clear and descriptive.
[x] Reviewers have been assigned to the pull request.
[x] Any potential security implications have been considered and addressed.
[x] Performance impact of the changes has been evaluated, if relevant.

If mobile/frontend pull request, what version of the backend is it stable, and running on?

Staging backend Because we still lack database refactoring, it can only be tested on the last section of the first course.

Notes for Reviewers

More tests to come later to day I promise 💯 👍

sofiiagran commented 10 months ago

@PrometheusBork now fixed

sofiiagran commented 10 months ago

@Donhusum can you please look at the implementation of getSection in StorageServices? does it look good/right?

sofiiagran commented 10 months ago

Snapshots fail... but I cannot update it on my computer...

Donhusum commented 10 months ago

I have written some comment on changes. I am also trying to test your branch on my device. I have some problems there, but i am usurer if it is due to the changes, or the fact that the DB have been semi purged.

It seem to have been problems do to the DB purge. I all works fine now.

sofiiagran commented 10 months ago

New commit in 20 min, waiting for DB to be restored, right now it is empty and cannot be tested

Educado-App / educado-mobile