Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RPMTranslator API #9

Merged
merged 58 commits into from
Mar 26, 2022
Merged

RPMTranslator API #9

merged 58 commits into from
Mar 26, 2022

Conversation

SiongSng
Copy link
Member

@SiongSng SiongSng commented Mar 12, 2022

To Do

Other

  • User role (translation manager)
  • Added all tests
  • Revirw

Export

  • Export general translation
  • Export patchouli book translation
  • Export custom text translation

Comment

  • Get
  • List by type and parent or reply comment
  • Add
  • Edit
  • Delete
  • Added tests

Glossary

  • Get
  • List
  • Add
  • Edit
  • Delete
  • Highlight

Translation

  • Get
  • List by translation content or language or translator
  • Add
  • Delete

Translation vote

  • List
  • Add
  • Edit
  • Cannel

Mod source info

  • Get
  • List by mod namespace/name
  • Add
  • Edit
  • Delete

Source file

  • Get
  • List
  • Add
  • Upload file from local
  • Edit
  • Delete

Source text

  • Get
  • List by source or key
  • Add
  • Edit
  • Delete

@SiongSng SiongSng added the enhancement New feature or request label Mar 12, 2022
@SiongSng SiongSng self-assigned this Mar 12, 2022
@SiongSng SiongSng linked an issue Mar 12, 2022 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Mar 12, 2022

Codecov Report

Merging #9 (93122b3) into main (f5266ff) will increase coverage by 18.90%.
The diff coverage is 91.35%.

@@             Coverage Diff             @@
##             main       #9       +/-   ##
===========================================
+ Coverage   69.18%   88.09%   +18.90%     
===========================================
  Files          31       50       +19     
  Lines        1496     2461      +965     
===========================================
+ Hits         1035     2168     +1133     
+ Misses        461      293      -168     
Impacted Files Coverage Δ
lib/utilities/data.dart 75.00% <ø> (+53.04%) ⬆️
lib/utilities/scam_detection.dart 100.00% <ø> (ø)
lib/data/user_view_count_filter.dart 13.63% <13.63%> (ø)
lib/utilities/utility.dart 14.54% <42.85%> (+8.66%) ⬆️
lib/database/models/auth/user_role_permission.dart 64.00% <64.00%> (ø)
lib/handler/auth_handler.dart 68.91% <66.66%> (+0.64%) ⬆️
lib/database/models/translate/translation.dart 69.44% <69.44%> (ø)
lib/routes/curseforge_route.dart 84.21% <71.42%> (ø)
lib/database/models/translate/mod_source_info.dart 76.92% <76.92%> (ø)
lib/routes/api_route.dart 80.00% <80.00%> (ø)
... and 46 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f5266ff...93122b3. Read the comment docs.

@SiongSng SiongSng marked this pull request as ready for review March 26, 2022 06:30
Copy link
Member Author

@SiongSng SiongSng left a comment

Choose a reason for hiding this comment

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

LGTM

@SiongSng SiongSng merged commit 0f6b3c6 into main Mar 26, 2022
@SiongSng SiongSng deleted the feature/translator branch March 26, 2022 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 2022 Q2
Development

Successfully merging this pull request may close these issues.

[Server] Support RPMTranslator
1 participant