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

Reentrancy anomaly on .forwardToCurrentFlow #175

Closed
erginbilgin opened this issue Jun 28, 2021 · 9 comments
Closed

Reentrancy anomaly on .forwardToCurrentFlow #175

erginbilgin opened this issue Jun 28, 2021 · 9 comments

Comments

@erginbilgin
Copy link

Using .forwardToCurrentFlow(withStep:) causes a reentrancy anomaly. It is observable on demo project as well, right after SettingsFlow:93 (Run demo app and tap on settings button).

@erginbilgin erginbilgin changed the title Reentrency anomaly on .forwardToCurrentFlow Reentrancy anomaly on .forwardToCurrentFlow Jun 28, 2021
@erginbilgin
Copy link
Author

@twittemb Are you still maintaining the project?

@twittemb
Copy link
Collaborator

twittemb commented Aug 2, 2021

@erginbilgin Hi, thanks for reaching. I'm still maintaining the project, I was on vacation with no computer ☺️.
I will check this issue ASAP.

@twittemb
Copy link
Collaborator

Hey @erginbilgin, could you give it another try on the branch feature/bump-deps (please perform a Carthage bootstrap because the deps have been bumped)

thanks.

@erginbilgin
Copy link
Author

@twittemb I will do at the first opportunity I find. 👍 Did you have a chance to check the other issue which I opened? #176

@twittemb
Copy link
Collaborator

@erginbilgin can you try the new 2.12.4 release please ?

@erginbilgin
Copy link
Author

It is not available yet in CocoaPods as I see. Will you push there soon?

@twittemb
Copy link
Collaborator

twittemb commented Aug 19, 2021

@erginbilgin

I've just released it :-)

@erginbilgin
Copy link
Author

erginbilgin commented Aug 20, 2021

@twittemb Sorry for delay. Just tested it and warnings are gone. Seems like #176 is solved too. Amazing. Thanks! :)

@twittemb
Copy link
Collaborator

Hey @erginbilgin

Happy to hear that.

Feel free to close the issues if you think they are resolved.

Bye.

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

No branches or pull requests

2 participants