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
Sc 10279/tenant update #55
Conversation
This pull request has been linked to Shortcut Story #10279: Tenant Resource Update API. |
Codecov Report
@@ Coverage Diff @@
## main #55 +/- ##
==========================================
- Coverage 47.48% 47.45% -0.04%
==========================================
Files 52 52
Lines 3637 3652 +15
==========================================
+ Hits 1727 1733 +6
- Misses 1714 1720 +6
- Partials 196 199 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
@bbengfort I resolved merged conflicts for this PR so the changes made in #51 also appear. You can review that one first and then take a look at this one. |
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.
Looks good! One small comment and it should be good to go!
Co-authored-by: Benjamin Bengfort <benjamin@rotational.io>
Scope of changes
Implementation of Tenant Update API definition on the client-side.
Fixes SC-10279
Type of change
Acceptance criteria
Describe how reviewers can test this change to be sure that it works correctly. Add a checklist if possible.
Author checklist
Reviewer(s) checklist