Skip to content
This repository was archived by the owner on Jan 14, 2026. It is now read-only.

Delete and Update actions + Refactor#6

Merged
infeeeee merged 4 commits intoinfeeeee:mainfrom
ihor-sokoliuk:main
Dec 31, 2024
Merged

Delete and Update actions + Refactor#6
infeeeee merged 4 commits intoinfeeeee:mainfrom
ihor-sokoliuk:main

Conversation

@ihor-sokoliuk
Copy link
Contributor

Added Delete and Update record actions to the library (in the end of the code changes).

Additionally, I formatted the scripts with a Black formatter.

@ihor-sokoliuk
Copy link
Contributor Author

@infeeeee feel free to review

@infeeeee
Copy link
Owner

Nice, I liked the delete_records method, why did you remove it? Cloud noco has some very strict rate limiting, it would be useful to delete a lot of records in a single call. It's just a list comprehension, I will do it.

@infeeeee infeeeee added the safe to test Test should run on this PR label Dec 30, 2024
@infeeeee infeeeee marked this pull request as draft December 30, 2024 21:58
@infeeeee infeeeee marked this pull request as ready for review December 30, 2024 21:58
@infeeeee infeeeee merged commit e7713df into infeeeee:main Dec 31, 2024
1 check passed
@infeeeee
Copy link
Owner

Messed up branches I will change some things

@infeeeee
Copy link
Owner

But thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

safe to test Test should run on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants