Skip to content
This repository was archived by the owner on Jan 29, 2026. It is now read-only.

feat(Recommendations): Rename operation to NewTabRecommendations#50

Merged
mmiermans merged 1 commit intomainfrom
fix-rename-recommendations-operation
Jun 5, 2023
Merged

feat(Recommendations): Rename operation to NewTabRecommendations#50
mmiermans merged 1 commit intomainfrom
fix-rename-recommendations-operation

Conversation

@mmiermans
Copy link
Copy Markdown
Contributor

@mmiermans mmiermans commented Jun 2, 2023

Goal

Give a more distinct, meaningful name to the operation that queries recommendations for NewTab, such that this operation can be distinguished in Apollo Studio and Sentry from other operations that query recommendations.

I'd love feedback/perspectives on:

  • Should the graphql filename be renamed as well for consistency?
  • Would folks prefer a different name?

References

JIRA ticket:

We use the operation name to identify operations in Apollo Studio,
and it would be helpful if this operation had a more distinct name.
@mmiermans mmiermans requested review from a team as code owners June 2, 2023 15:50
@mmiermans mmiermans requested review from Herraj and cmharlow June 2, 2023 15:50
@mmiermans mmiermans merged commit 9f00f7c into main Jun 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants