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

feat: Clear config cache in start_io() #6228

Merged
merged 2 commits into from
Nov 19, 2024
Merged

feat: Clear config cache in start_io() #6228

merged 2 commits into from
Nov 19, 2024

Conversation

Hocuri
Copy link
Collaborator

@Hocuri Hocuri commented Nov 19, 2024

This is needed for iOS, see comment in the code. An alternative would be to add an API invalidate_config_cache() or to do nothing and just assume that things will be fine.

We could mention this in the doc comment of start_io(), but right now this PR does not, since it's only important for one platform.

TODO:
- adapt start_io() documentation
- test?

This is needed for iOS, see comment in the code. An alternative would be
to add an API `invalidate_config_cache()` or to do nothing and just
assume that things will be fine.
@r10s
Copy link
Member

r10s commented Nov 19, 2024

ftr, this is the PR that calls startIo delayed on mainapp, after the NSE process has eventually terminated: deltachat/deltachat-ios#2393

@Hocuri Hocuri enabled auto-merge (squash) November 19, 2024 15:47
@Hocuri Hocuri merged commit c181db6 into main Nov 19, 2024
38 checks passed
@Hocuri Hocuri deleted the hoc/clear-config-cache branch November 19, 2024 15:59
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