rage / mooc.fi

https://mooc.fi
5 stars 6 forks source link

Feat: add extra progress to summary; nested summaries #1124

Closed mipyykko closed 1 year ago

mipyykko commented 1 year ago

Allow for closer (=typed) querying of the extra field that stores some information on user progress on tiered courses. Previously it was set as a Json type.

Allow for nested user course summaries -- if a course has tiers, then the topmost handler course will have the tiers as its children. Summary view also shows these as nested.

codecov[bot] commented 1 year ago

Codecov Report

Merging #1124 (c179630) into master (91bfd9e) will decrease coverage by 1.06%. The diff coverage is 47.36%.

:exclamation: Current head c179630 differs from pull request most recent head 23d3bb3. Consider uploading reports for the commit 23d3bb3 to get more accurate results

@@            Coverage Diff             @@
##           master    #1124      +/-   ##
==========================================
- Coverage   66.57%   65.52%   -1.06%     
==========================================
  Files         122      124       +2     
  Lines        4976     5221     +245     
  Branches     1107     1158      +51     
==========================================
+ Hits         3313     3421     +108     
- Misses       1548     1676     +128     
- Partials      115      124       +9     
Flag Coverage Δ
backend 65.52% <47.36%> (-1.06%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
backend/api/routes/abStudio/abEnrollments.ts 90.47% <ø> (ø)
backend/api/routes/progress.ts 31.76% <ø> (ø)
...nsumer/common/userCourseProgress/BAI/completion.ts 14.70% <ø> (ø)
...Consumer/common/userCourseProgress/BAI/progress.ts 84.48% <ø> (ø)
...afkaConsumer/common/userCourseProgress/saveToDB.ts 86.95% <ø> (ø)
...end/bin/kafkaConsumer/exerciseConsumer/saveToDB.ts 92.30% <ø> (ø)
backend/graphql/Completion/index.ts 100.00% <ø> (ø)
backend/graphql/Completion/model.ts 40.74% <0.00%> (-1.04%) :arrow_down:
backend/graphql/Completion/mutations.ts 56.47% <ø> (ø)
backend/graphql/CompletionRegistered.ts 45.31% <0.00%> (ø)
... and 27 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more