-
Notifications
You must be signed in to change notification settings - Fork 32
Implemented Task Tags Endpoints #66
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
Conversation
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
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, thanks for adding tags endpoints! Just left nit comments for readme formatting.
Co-authored-by: Fatih Kurtoglu <fatih.kurtoglu@scale.com>
Co-authored-by: Fatih Kurtoglu <fatih.kurtoglu@scale.com>
Co-authored-by: Fatih Kurtoglu <fatih.kurtoglu@scale.com>
Co-authored-by: Fatih Kurtoglu <fatih.kurtoglu@scale.com>
Co-authored-by: Fatih Kurtoglu <fatih.kurtoglu@scale.com>
Co-authored-by: Fatih Kurtoglu <fatih.kurtoglu@scale.com>
Co-authored-by: Fatih Kurtoglu <fatih.kurtoglu@scale.com>
Co-authored-by: Fatih Kurtoglu <fatih.kurtoglu@scale.com>
Co-authored-by: Fatih Kurtoglu <fatih.kurtoglu@scale.com>
Co-authored-by: Fatih Kurtoglu <fatih.kurtoglu@scale.com>
@fatihkurtoglu apparently we have to use Opal now to request for write access to this repo 😢 |
Pull Request Summary
Description
Please include a summary of the changes. Please also include the relevant context and motivation. List any dependencies and assumptions that are required for this change.
Implemented task tags endpoints to set, add, and delete tags from a given task.
How did you test your code?
Which of the following have you done to test your changes? Please describe the tests that you ran to verify your changes.
tests/
for the newly implemented methodstests/
to cover changes made to existing methodsChecklist
Please make sure all items in this checklist have been fulfilled before sending your PR out for review!
_version.py
(unless this PR only updates the documentation).