-
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
Upgrade routine for old Stream Settings records #328
Comments
@powelski Can you take this one? |
@c3mdigital We can just get the routine API done and then release the routine for this later. It might even be a moot point now since this issue is almost a month old. Still, I'd probably like to do it to ensure 100% backwards compat with old data that might exist in someone's Stream data. |
@c3mdigital Yes, please go ahead and include an update routine for these as part of #379. |
This is so old that it really no longer applies. Closing. |
Since we now have improved action links for Stream Settings (see #305) we need to update past entries by giving them an
option_key
meta value.We can do this by selecting all
stream_meta
that has anoption
key equal towp_stream
and then use thelabel
meta to reference the settings fields array to determine theoption_key
./cc @powelski
The text was updated successfully, but these errors were encountered: