Update terminology from "production endpoint" to "main endpoint"#629
Update terminology from "production endpoint" to "main endpoint"#629Thushani-Jayasekera wants to merge 1 commit intowso2:mainfrom
Conversation
WalkthroughThe pull request updates terminology across two files, replacing "production endpoint" references with "main endpoint" in the GraphQL schema viewer component's accessibility label and the API documentation template. No functional, structural, or behavioral changes were introduced. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
libs/graphql-schema-viewer/src/components/GraphQLSchemaViewer/GraphQLSchemaViewer.tsx (1)
305-332:⚠️ Potential issue | 🟡 MinorUpdate visible label to match aria-label and sandbox pattern.
The
aria-labelreads"Copy main endpoint URL to clipboard"(line 305), but the visible label still reads"Endpoint"(line 332). Update it to"Main Endpoint"to align with the accessibility label and match the pattern used for the Sandbox card.Suggested fix
<div style={{ ...styles.endpointLabel, ...styles.endpointLabelProduction }}> - Endpoint + Main Endpoint </div>🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@libs/graphql-schema-viewer/src/components/GraphQLSchemaViewer/GraphQLSchemaViewer.tsx` around lines 305 - 332, The visible label text inside the production endpoint card currently reads "Endpoint" but the aria-label is "Copy main endpoint URL to clipboard" and the sandbox card uses "Main Endpoint"; update the rendered label string to "Main Endpoint" where styles.endpointLabel and styles.endpointLabelProduction are applied (the same JSX block that calls handleEndpointCopy and sets aria-label) so the UI text matches the aria-label and sandbox pattern.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Outside diff comments:
In
`@libs/graphql-schema-viewer/src/components/GraphQLSchemaViewer/GraphQLSchemaViewer.tsx`:
- Around line 305-332: The visible label text inside the production endpoint
card currently reads "Endpoint" but the aria-label is "Copy main endpoint URL to
clipboard" and the sandbox card uses "Main Endpoint"; update the rendered label
string to "Main Endpoint" where styles.endpointLabel and
styles.endpointLabelProduction are applied (the same JSX block that calls
handleEndpointCopy and sets aria-label) so the UI text matches the aria-label
and sandbox pattern.
Purpose
$subject
Summary by CodeRabbit