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

api: remove unused dc_accounts_all_work_done() #5384

Merged
merged 2 commits into from
Mar 30, 2024
Merged

Conversation

r10s
Copy link
Member

@r10s r10s commented Mar 27, 2024

it was used by iOS to know when a background fetch was complete; meanwhile the superiour dc_accounts_background_fetch() is used for that.

there is still the corresponding context function dc_all_work_done(), this not used by any UI as well, however, it is in use by a python tests.

not sure, what to do with it, at a first glance, the test still seems useful.

@r10s r10s requested review from link2xt and iequidoo March 27, 2024 16:19
@r10s r10s force-pushed the r10s/remove-all-work-done branch from 3572cdc to 0d57dff Compare March 27, 2024 16:20
Copy link
Collaborator

@iequidoo iequidoo left a comment

Choose a reason for hiding this comment

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

Commit message header probably should include refactor: as per CONTRIBUTING.md

@r10s r10s changed the title remove unused dc_accounts_all_work_done() refactor: remove unused dc_accounts_all_work_done() Mar 27, 2024
it was used by iOS to know when a background fetch was complete;
meanwhile the superiour dc_accounts_background_fetch() is used for that.

there is still the corresponding context function dc_all_work_done(),
this not used by any UI as well, however, it is in use by a python tests.
@r10s r10s force-pushed the r10s/remove-all-work-done branch from 0d57dff to ecff0c9 Compare March 27, 2024 21:33
@r10s r10s changed the title refactor: remove unused dc_accounts_all_work_done() api: remove unused dc_accounts_all_work_done() Mar 27, 2024
@r10s
Copy link
Member Author

r10s commented Mar 27, 2024

i think, it is an API change, anyways, thanks for reminding to the convention!

@link2xt
Copy link
Collaborator

link2xt commented Mar 27, 2024

CI fails because node/src/module.c still has a reference to dc_accounts_all_work_done().

@r10s r10s merged commit 7bf9c4a into main Mar 30, 2024
38 checks passed
@r10s r10s deleted the r10s/remove-all-work-done branch March 30, 2024 00:18
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.

3 participants