-
-
Notifications
You must be signed in to change notification settings - Fork 277
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
MBS-13560: Sync Entity::Artist
with beta
#3252
Conversation
Thanks, LGTM. |
`Entity::Artist` is missing a couple Moose attributes that were added to the beta code. Entities that are cached on production but loaded on beta will be missing these, and cause beta to crash. The relevant changes have been synced from d981ba8.
I forgot we also need the part in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTMBDNT
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same! ⛵
For the record, it refers to the pull request #3191 currently deployed in beta.
* production: MBS-13560: Sync `Entity::Artist` with beta (#3252) Amend e5776e7: Fix testing four-byte characters Amend e5776e7: Improve phrasing by reosarevok MBS-13562 (III): Check editing unlimited strings MBS-13562 (II): Set indexable string size to 2704 MBS-13562 (I): Set one-line string length to 1024
* master: Update POT files using the production database MBS-13560: Sync `Entity::Artist` with beta (#3252) Amend e5776e7: Fix testing four-byte characters Amend e5776e7: Improve phrasing by reosarevok MBS-13562 (III): Check editing unlimited strings MBS-13562 (II): Set indexable string size to 2704 MBS-13562 (I): Set one-line string length to 1024 MBS-13533: Fix order of cover art in release group artwork (#3238)
Problem
MBS-13560
Entity::Artist
is missing a couple Moose attributes that were added to the beta code. Entities that are cached on production but loaded on beta will be missing these, and cause beta to crash.Solution
The relevant changes have been synced from d981ba8.
Testing
Just the existing automated tests.