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

Add warning on save export page when exceeding dragon limit #796

Merged
merged 1 commit into from
May 13, 2024

Conversation

SapiensAnatis
Copy link
Owner

This is based on player feedback from the changes to save import in #792 to provide a more clear warning about what will happen when importing a save with an excess number of dragons. Since we are not currently able to show feedback on the save import process, we will settle for introducing a dialog in the save export page where most players who end up reimporting their save will have to go first.

This is based on player feedback from the changes to save import in
#792 to provide a more clear warning about what will happen when
importing a save with an excess number of dragons. Since we are not
currently able to show feedback on the save import process, we will
settle for introducing a dialog in the save export page where most
players who end up reimporting their save will have to go first.
@SapiensAnatis SapiensAnatis merged commit e1d7b77 into main May 13, 2024
7 checks passed
@SapiensAnatis SapiensAnatis deleted the feature/website-warning branch May 13, 2024 21:16
SapiensAnatis added a commit that referenced this pull request Jun 30, 2024
This is based on player feedback from the changes to save import in #792
to provide a more clear warning about what will happen when importing a
save with an excess number of dragons. Since we are not currently able
to show feedback on the save import process, we will settle for
introducing a dialog in the save export page where most players who end
up reimporting their save will have to go first.
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.

1 participant