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

Fix flowDidChange being called extraneously #850

Merged
merged 4 commits into from
Jan 25, 2024

Conversation

Sjmarf
Copy link
Contributor

@Sjmarf Sjmarf commented Jan 11, 2024

flowDidChange was called via onAppear on the Window view. onAppear was triggered when navigating back from an external link, which called flowDidChange. I've fixed this by only calling flowDidChange the first time onAppear is triggered.

This might be related to the app store review about having to re-authenticate when navigating back from an external link?

It might also be related to the error haptic that I've been getting when navigating back from a link, though I'm not sure because I can't reproduce that bug at the moment. It might be dependent on internet speed or something, because I was able to reproduce it consistently yesterday and nothing's changed since then.

@Sjmarf Sjmarf requested a review from a team as a code owner January 11, 2024 21:53
@Sjmarf Sjmarf requested review from mormaer and ShadowJonathan and removed request for a team January 11, 2024 21:53
@Sjmarf Sjmarf added the bug Something isn't working label Jan 11, 2024
@Sjmarf Sjmarf enabled auto-merge (squash) January 11, 2024 21:58
Copy link
Member

@EricBAndrews EricBAndrews left a comment

Choose a reason for hiding this comment

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

Looks good! Maybe leftover test print?

Mlem/Window.swift Outdated Show resolved Hide resolved
Copy link
Member

@EricBAndrews EricBAndrews left a comment

Choose a reason for hiding this comment

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

Nice 👍

@Sjmarf Sjmarf merged commit 25103de into dev Jan 25, 2024
4 checks passed
@Sjmarf Sjmarf deleted the sjmarf/flow-change-onappear-fix branch January 25, 2024 23:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants