-
Notifications
You must be signed in to change notification settings - Fork 21
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
Added public/private editing #67
Conversation
There are conflicts to be resolved |
Codecov Report
@@ Coverage Diff @@
## ign-fuel-tools4 #67 +/- ##
===================================================
- Coverage 78.50% 76.60% -1.90%
===================================================
Files 16 16
Lines 2214 2278 +64
===================================================
+ Hits 1738 1745 +7
- Misses 476 533 +57
Continue to review full report at Codecov.
|
Signed-off-by: Nate Koenig <[email protected]>
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.
LGTM. The only suggestion I have is documenting the new command on the README and tutorials.
Signed-off-by: Nate Koenig <[email protected]>
Updated tutorial: b4a0925 |
Add support for editing models via a PATCH call. I'm limiting the scope of this PR to just the public/private field in models.