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

[Testing] Altoholic v0.0.0.11 #4518

Merged
merged 1 commit into from
Sep 1, 2024
Merged

[Testing] Altoholic v0.0.0.11 #4518

merged 1 commit into from
Sep 1, 2024

Conversation

Sohtoren
Copy link
Contributor

  • Migrate database provider from litedb to sqlite
    • Data should migrate without issue however in the case it doesn't, disable the plugin and manually delete the current database file in the plugin config folder before re-enabling.

@bleatbot bleatbot enabled auto-merge (squash) August 31, 2024 12:51
@bleatbot
Copy link
Collaborator

Take care! Please test your plugins in-game before submitting them here to prevent crashes and instability. We really appreciate it!

The average merge time for plugin updates is currently 5 hours.

✔️Altoholic [testing-live] - 296c80a - v0.0.0.11 - Diff (1314 lines, prev. 0.0.0.10) - Semantic

Show log - Review

@bleatbot bleatbot added the size-large Diff for this PR is large. label Aug 31, 2024
@philpax
Copy link
Contributor

philpax commented Aug 31, 2024

Will users lose their data if the migration fails?

@Sohtoren
Copy link
Contributor Author

Will users lose their data if the migration fails?

Not per se, the migration will try again the next time the plugin is loaded.

It's mostly for the unused currency history, characters data will be the same next login on each anyway.

@bleatbot bleatbot merged commit 08422fc into goatcorp:main Sep 1, 2024
2 checks passed
@Sohtoren Sohtoren deleted the altoholic branch September 1, 2024 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size-large Diff for this PR is large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants