-
Notifications
You must be signed in to change notification settings - Fork 116
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
Create a separate API for handling DB update routines #379
Comments
@fjarrett It might be beyond the scope of this issue but what about having a export / import settings routine? This would just be a json dump of all the |
@c3mdigital Yeah that's a cool idea. And definitely should be a separate issue as this pertains only to routines we need to do after plugin updates. |
@c3mdigital And yes, you can completely uninstall Stream including it's tables and all its options in the DB. That was done in #102 and #114. |
@c3mdigital I'm getting an error when attempting to update the DB.
Based on our convo last week, I think this is because it's looking for an option named |
Our current method for dealing with DB update routines is starting to get a little messy, and some migration routines that require Connectors already be registered are needing their own method outside of the main
update()
method just to be able to fire at the correct time.Furthermore, we should be asking users to opt-in to these updates, rather than just automatically doing it on reactivation. This is always a good idea just so admins know when their DB is being altered. This step is even more critical for very large sites, so a snapshot can be taken beforehand.
To remedy this, we should create a separate API specifically for handling DB update routines and prompt users to take action when updates are required. Once the update is performed, or if there are no routines to run, then the
stream_db
option can be updated.UPDATE REQUIRED MESSAGE
SUCCESS MESSAGE
The text was updated successfully, but these errors were encountered: