-
Notifications
You must be signed in to change notification settings - Fork 153
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
feat(onyx-913): use new consignmentSubmission MP fields when rendering submission state section #13849
Conversation
…g submission state section
STATUSES[displayText.toLowerCase()]?.description || "" | ||
let stateLabelColor = "yellow150" | ||
if (["APPROVED", "REJECTED", "CLOSED", "PUBLISHED"].includes(state)) | ||
stateLabelColor = "orange150" |
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.
@joeyAghion you said that there are no submissions in "sold" state anymore - this we the only one with black100
color. It's ok that I do not support it anymore, right?
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.
Correct. That isn't a valid submission state, but came from the consigned_partner_submission
that hasn't been in use recently.
STATUSES[displayText.toLowerCase()]?.description || "" | ||
let stateLabelColor = "yellow150" | ||
if (["APPROVED", "REJECTED", "CLOSED", "PUBLISHED"].includes(state)) | ||
stateLabelColor = "orange150" |
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.
Correct. That isn't a valid submission state, but came from the consigned_partner_submission
that hasn't been in use recently.
The type of this PR is: Feat
This PR solves ONYX-913
Description
Parent Jira ticket with more details
Force/Eigen hardcode submission status labels and help messages. We've moved this logic to
myCollectionConnection...consignmentSubmission
MP field (artsy/metaphysics#5702) and in this PR we are replacing old "hardcoded" data with the new one taken from MP. Nothing should change visually, so I am not attaching screenshots.