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.

Sign up for GitHub

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

Jump to bottom

feat: env variable for infinite discovery use internal tracking of seen artworks #6454

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat: env variable for infinite discovery use internal tracking of seen artworks #6454

wants to merge 1 commit into from

Conversation

nickskalkin
Copy link
Contributor

Allows dynamically control behaviour of discoverArtworks endpoint—this should give us more flexibility when switching from storage-based tracking of seen artworks to backend-based tracking.

@nickskalkin nickskalkin self-assigned this Feb 20, 2025
@artsy-peril
Copy link
Contributor

artsy-peril bot commented Feb 20, 2025


Warnings
⚠️ The V2 schema in this PR has breaking changes with Force. Remember to update the Force schema if necessary.

Argument 'useInternalTracking: Boolean' was removed from field 'Query.discoverArtworks'
Argument 'useInternalTracking: Boolean' was removed from field 'Viewer.discoverArtworks'

Generated by 🚫 dangerJS against 5d777f6

@nickskalkin nickskalkin mentioned this pull request Feb 20, 2025
Open
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nickskalkin nickskalkin

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nickskalkin