Skip to content
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: switch fairs landing query to MP v2 #6387

Merged
merged 1 commit into from
Oct 1, 2020

Conversation

anandaroop
Copy link
Member

https://artsyproduct.atlassian.net/browse/FX-2329

Keeping it simple, just changing the query, and aliasing fields as necessary.

@anandaroop anandaroop self-assigned this Oct 1, 2020
fairs: items {
... on FeaturedLinkItem {
... on FeaturedLink {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure I fully followed the change btw v1 and v2 when it comes to this type, but this does work.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sweet! I think @dzucconi may have done some of the V2 updates for featured links? I don't fully recall these ones.

@anandaroop anandaroop assigned mzikherman and unassigned anandaroop Oct 1, 2020
@mzikherman mzikherman merged commit 3a8f70d into master Oct 1, 2020
@artsy-peril artsy-peril bot mentioned this pull request Oct 1, 2020
dblandin added a commit that referenced this pull request Oct 6, 2020
Quick follow-up to #6387 (FX-2329) to
also migrate the client code to use Metaphysics V2.
artsy-peril bot pushed a commit that referenced this pull request Oct 6, 2020
Quick follow-up to #6387 (FX-2329) to
also migrate the client code to use Metaphysics V2.
@zephraph zephraph deleted the roop/FX-2329-fairs-landing-mp-v2 branch October 23, 2020 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants