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

Data loss can occur #62

Closed
6 tasks done
bitbrain opened this issue Aug 13, 2023 · 2 comments · Fixed by #87
Closed
6 tasks done

Data loss can occur #62

bitbrain opened this issue Aug 13, 2023 · 2 comments · Fixed by #87
Labels
🐛 bug Something isn't working 🗃 datastore Datastore functionality.
Milestone

Comments

@bitbrain
Copy link
Owner

bitbrain commented Aug 13, 2023

It appears that sometimes it can happen that a bad save would remove all entities (but not categories). Only resetting to a previous version solves this issue at the moment.

Addressed Cases

@bitbrain bitbrain added 🐛 bug Something isn't working 🗃 datastore Datastore functionality. labels Aug 13, 2023
@bitbrain bitbrain added this to the 1.0.0 milestone Aug 13, 2023
@bitbrain bitbrain pinned this issue Aug 21, 2023
@bitbrain bitbrain unpinned this issue Aug 26, 2023
@bitbrain bitbrain reopened this Aug 26, 2023
@bitbrain
Copy link
Owner Author

This is still happening in 1.0alpha2 - especially when upgrading Pandora and then saving.

@bitbrain bitbrain pinned this issue Aug 26, 2023
@bitbrain
Copy link
Owner Author

bitbrain commented Sep 1, 2023

In case there has been any bug/issue with parsing PandoraEntity, rather than not loading it (current behavior) we should "lock" the save mechanism and only unblock it after the addon has been "reloaded" - that ideally should happen automatically once the error is fixed. We would need to figure out how exactly Godot notifies on compilation errors.

  1. attempt to load Pandora
  2. in case loading failed for whatever reason, prevent being able to save (and even use the Pandora UI) but instead retry to load
  3. once the compilation errors have been addressed and all entities could be loaded correctly, display the Pandora UI

Once data is loaded successfully, someone could go and again break things - in case trying to save an existing entity results in either a null or empty dictionary, do the same approach:

  1. disable the save mechanism until the issue is fixed (and notify the user)
  2. once the compilation/error is resolved, auto-save

@bitbrain bitbrain changed the title Bad save can remove all entities Data loss can occur Sep 1, 2023
@bitbrain bitbrain closed this as completed Sep 2, 2023
@bitbrain bitbrain unpinned this issue Sep 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working 🗃 datastore Datastore functionality.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant