Skip to content
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

Adds delete_transaction_categories() and delete_transaction_category() #62

Merged
merged 1 commit into from
Jan 5, 2024

Conversation

hammem
Copy link
Owner

@hammem hammem commented Jan 5, 2024

Adds methods to enable deleting of one or more categories. Was able to test out a few different scenarios to ensure it's working as expected.

Example categories at start
Example categories at start

Results of API calls in testing
Results of API calls in testing

Deletion of a non-existent category results in an exception, as expected.

Note: It appears 're-deleting' returns success from the API, but that may just be a timing / sync issue.

Categories after single and multi-delete operations
Categories after deletion

After tests, the Hotels, Airlines, and Entertainment categories are all gone.

@hammem hammem linked an issue Jan 5, 2024 that may be closed by this pull request
@hammem hammem requested a review from grablair January 5, 2024 02:33
@grablair grablair merged commit 658d32a into main Jan 5, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add methods to delete Categories
2 participants