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

Switching floors in 'try' functionality loses all current mapping progress #150

Closed
Wotuu opened this issue Mar 4, 2019 · 0 comments

Comments

Projects
None yet
1 participant
@Wotuu
Copy link
Owner

commented Mar 4, 2019

This is kind of bad, so perhaps a better approach to the 'try' functionality is to save all results to a temporary dungeon route that may be removed after a day or so. This allows people to register while on that page and then save the route they worked on as well. Reduces complexity in the code as well.

@Wotuu Wotuu added the bug label Mar 4, 2019

@Wotuu Wotuu changed the title Switching floors in 'try' functionality loses all current progress. Switching floors in 'try' functionality loses all current mapping progress Mar 4, 2019

Wotuu added a commit that referenced this issue Mar 8, 2019

Wotuu added a commit that referenced this issue Mar 10, 2019

#150 Try mode now actively saves information to the server into a rou…
…te. This has the advantage of keeping progress when switching floors (was a bug). I've also added options to go from a try mode to directly registering and logging in. When logged in, you can claim the route to make it appear in your routes overview. Any unclaimed try routes are deleted after a day (not yet implemented).

Wotuu added a commit that referenced this issue Mar 11, 2019

@Wotuu Wotuu closed this Mar 11, 2019

Wotuu added a commit that referenced this issue Apr 5, 2019

#150 Non-logged in users can now properly make use of the try functio…
…nality. Will have to verify the safety of creating a new route though, but I think it should still be secure.

Wotuu added a commit that referenced this issue Apr 9, 2019

#150 Fixed some issues with the new try functionality and expiring ro…
…utes.

Added a scheduled thing that deletes any routes that are expired.
Fixed an issue that prevented routes from showing up in the routes page.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.