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

General bug fixes #22

Merged
merged 3 commits into from Dec 7, 2017
Merged

General bug fixes #22

merged 3 commits into from Dec 7, 2017

Conversation

dlmr
Copy link
Owner

@dlmr dlmr commented Dec 6, 2017

Fixes some bugs, see commit messages.

This would be a problem when loading slowly and clicking a new link while the other is still loading.
It could call onCompleted for afterTransition twice without this fix.
@mbjurman
Copy link

mbjurman commented Dec 6, 2017

LGTM, what would @PAkerstrand say?

@dlmr dlmr merged commit a2555a7 into master Dec 7, 2017
@dlmr dlmr deleted the fix/solve-bugs branch December 7, 2017 18:58
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.

None yet

2 participants