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

Add method batch_update_list #4

Merged
merged 25 commits into from
Feb 26, 2024
Merged

Add method batch_update_list #4

merged 25 commits into from
Feb 26, 2024

Conversation

tr4nt0r
Copy link
Collaborator

@tr4nt0r tr4nt0r commented Feb 12, 2024

Adds a method batch_update_list.
This uses the same endpoint as the Bring App for manipulating the lists and supports adding, completing and removing items from a list. Using the items uuid as unique identifier is supported with this method, so differentiating items with the same name but with different specifications is possible. Changing an items name is also possible but should be avoided, as it has weird effects in the App.

Copy link
Owner

@miaucl miaucl left a comment

Choose a reason for hiding this comment

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

Add an example to the README for it as well, showing the name can be changed and adding a comment, that it might be "buggy" in the list on the app.

bring_api/types.py Show resolved Hide resolved
@miaucl
Copy link
Owner

miaucl commented Feb 14, 2024

Could you also add this endpoint to the test script? Even though the test workflow does not work atm :)

@tr4nt0r
Copy link
Collaborator Author

tr4nt0r commented Feb 14, 2024

Could you also add this endpoint to the test script? Even though the test workflow does not work atm :)

already did

method name changed to better reflect the method name used in the Bring android
app `batchUpdateBringList`
@tr4nt0r tr4nt0r changed the title Add method change_list Add method batch_udate_list Feb 15, 2024
@tr4nt0r
Copy link
Collaborator Author

tr4nt0r commented Feb 15, 2024

renamed method to better reflect the method name used in the Bring Android app (batchUpdateBringList)

@tr4nt0r tr4nt0r marked this pull request as draft February 15, 2024 15:07
@tr4nt0r tr4nt0r marked this pull request as ready for review February 15, 2024 16:47
@miaucl miaucl changed the title Add method batch_udate_list Add method batch_update_list Feb 15, 2024
@miaucl miaucl self-requested a review February 15, 2024 17:20
@miaucl
Copy link
Owner

miaucl commented Feb 15, 2024

@tr4nt0r What is currently not possible with the batch_update_method?

@tr4nt0r
Copy link
Collaborator Author

tr4nt0r commented Feb 15, 2024

@tr4nt0r What is currently not possible with the batch_update_method?

i think it's feature complete, it can do everything the app can do. It can do the same as saveItem/completeItem/updateItem/remoteItem methods.

@tr4nt0r tr4nt0r self-assigned this Feb 26, 2024
@tr4nt0r tr4nt0r added the 🚀 feature New feature or request label Feb 26, 2024
Add support for item_uuid to `save_item`, `update_item`, `complete_item` and `remove_item`.
Refactor methods to wrapper methods for batch_update_list for item_uuid support.
README.md Show resolved Hide resolved
@tr4nt0r tr4nt0r merged commit 161b5c0 into miaucl:main Feb 26, 2024
3 checks passed
@tr4nt0r tr4nt0r deleted the change_list branch February 26, 2024 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants