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
refactor: Migrate update and delete of lists to Tapir #3013
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3013 +/- ##
===========================================
+ Coverage 11.74% 88.42% +76.68%
===========================================
Files 246 260 +14
Lines 22907 22951 +44
===========================================
+ Hits 2690 20295 +17605
+ Misses 20217 2656 -17561 ☔ View full report in Codecov by Sentry. |
PUT /admin/lists/**
to tapir
PUT /admin/lists/**
to tapir6917e63
to
9d9a8a2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job, thanks! Only one suggestion below:
Pull Request Checklist
Task Description/Number
based on #3012 which has to be merged first.
Migrates update and delete related
/admin/lists
endpoints to Tapir, includingPUT /admin/lists/**
DELETE /admin/lists/**
PR Type
Basic requirements for bug fixes and features
Does this PR introduce a breaking change?
Does this PR change client-test-data?