rage / mooc.fi

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

Feat: course tag edit #1109

Closed mipyykko closed 1 year ago

mipyykko commented 1 year ago

Can add and delete course tags to courses in the new editor. Ability to actually edit the tags not implemented, has to be done manually for now. Also changes the tag DB implementation a bit so the tag ID is a string instead of an UUID.

codecov[bot] commented 1 year ago

Codecov Report

Merging #1109 (ff43210) into master (7454bd6) will decrease coverage by 0.10%. The diff coverage is 40.00%.

@@            Coverage Diff             @@
##           master    #1109      +/-   ##
==========================================
- Coverage   67.49%   67.39%   -0.10%     
==========================================
  Files         118      118              
  Lines        4744     4748       +4     
  Branches     1018     1020       +2     
==========================================
- Hits         3202     3200       -2     
- Misses       1425     1431       +6     
  Partials      117      117              
Flag Coverage Δ
backend 67.37% <40.00%> (-0.10%) :arrow_down:

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

Impacted Files Coverage Δ
backend/tests/data/fixtures.ts 100.00% <ø> (ø)
backend/tests/util/jest.ts 74.60% <ø> (-1.59%) :arrow_down:
backend/graphql/Tag.ts 48.59% <14.28%> (-1.89%) :arrow_down:
backend/graphql/CourseTag.ts 37.33% <100.00%> (ø)
backend/graphql/TagTranslation.ts 78.57% <100.00%> (ø)

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