fix(samples): CAI-8463 capitalize App in React app heading for copy consistency - #740
Conversation
|
This pull request is automatically being deployed by Amplify Hosting (learn more). |
| <IconProvider iconSet="momentum-icons"> | ||
| <div className="webexTheme"> | ||
| <h1>Contact Center Widgets in a React app</h1> | ||
| <h1>Contact Center Widgets in a React App</h1> |
There was a problem hiding this comment.
Let's make it Contact center widgets in a react app
| <IconProvider iconSet="momentum-icons"> | ||
| <div className="webexTheme"> | ||
| <h1>Contact Center Widgets in a React app</h1> | ||
| <h1>Contact center widgets in a react app</h1> |
There was a problem hiding this comment.
Actaully, let's change it back to Contact Center Widgets in a React app
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cb697b14cf
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| <IconProvider iconSet="momentum-icons"> | ||
| <div className="webexTheme"> | ||
| <h1>Contact Center Widgets in a React app</h1> | ||
| <h1>Contact center widgets in a react app</h1> |
There was a problem hiding this comment.
The rendered heading now says react app, so the requested capitalization of App is not applied and the proper name React is newly lowercased; Contact Center Widgets is also lowercased unexpectedly. This leaves the sole copy fix ineffective and contradicts the existing page title in public/index.html, which uses Contact Center Widgets in a React App.
Useful? React with 👍 / 👎.
4 participants
COMPLETES #CAI-8463
This pull request addresses
Copy inconsistency in the sample React app heading:
React appwas written with a lowercase "app", which doesn't match standard capitalization for "React App".by making the following changes
Updated the
<h1>heading inwidgets-samples/cc/samples-cc-react-app/src/App.tsxfromContact Center Widgets in a React apptoContact Center Widgets in a React App.Change Type
The following scenarios were tested
Visual change only — the heading text capitalization was corrected. No functional logic was modified.
The GAI Coding Policy And Copyright Annotation Best Practices
Checklist before merging