-
Notifications
You must be signed in to change notification settings - Fork 152
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
chore(alerts): clean up onyx_artwork_alert_modal_v2
flag and dead code
#13426
Conversation
5282c56
to
55589b2
Compare
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.
looks good to me! Also tested on review app briefly and all seems working
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.
⭐⭐⭐
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.
⭐⭐⭐
Too good to be approved once 😄 |
Haha thanks! I'll rebase and merge now. |
test: fix mocking for tracking test: update spec to fetch pill data from MP preview rather than artwork
test: update spec to use new edit form
55589b2
to
07327eb
Compare
The type of this PR is: Chore
This PR solves ONYX-680
Review app: https://onyx-680.artsy.net
Description
Now that v2 alerts have been out in the wild for a while, we are starting to clean up feature flags and dead code.
This PR starts that process with the
onyx_artwork_alert_modal_v2
feature flag.Tests are passing; but for extra confidence that this does not introduce any regressions feel free to try out https://onyx-680.artsy.net
(h/t @tam-kis for pairing on the test fixes!)