Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Project Features incl update/delete #73

Merged
merged 11 commits into from
Apr 20, 2020

Conversation

tombuildsstuff
Copy link
Contributor

This PR includes & rebases the work done by @JanBerktold in #63 - but also includes support for Updating and Deleting them

For the sake of completeness I've added a couple of update tests too, toggling most of the fields in the Project Feature to ensure they're updated

@tombuildsstuff tombuildsstuff mentioned this pull request Apr 13, 2020
Copy link
Owner

@cvbarros cvbarros left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @tombuildsstuff , thanks for the PR! 🎉
I've added a minor question about the introduction of a dependency - That cleared, I'll get this merged and released asap 👍

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants