-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Fix team edit API panic #6780
Fix team edit API panic #6780
Conversation
Thanks for adding tests 👍 small nit but otherwise LGTM |
Codecov Report
@@ Coverage Diff @@
## master #6780 +/- ##
==========================================
+ Coverage 40.99% 41.22% +0.22%
==========================================
Files 421 421
Lines 58054 58054
==========================================
+ Hits 23802 23933 +131
+ Misses 31086 30954 -132
- Partials 3166 3167 +1
Continue to review full report at Codecov.
|
Co-Authored-By: ngourdon <[email protected]>
@ngourdon thanks for PR :) would you be able to backport this PR to the release/v1.8 branch? |
fix #6777 and add tests for team API