Update heading text on Users page for testing purposes#25
Update heading text on Users page for testing purposes#25kasperpeulen wants to merge 1 commit intomainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This pull request updates the heading text on the Users page by appending "Test" to the existing "Epic Notes Users" heading, presumably for testing purposes as indicated in the PR title.
- Modified the main heading text from "Epic Notes Users" to "Epic Notes Users Test"
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| return ( | ||
| <div className="container mt-36 mb-48 flex flex-col items-center justify-center gap-6"> | ||
| <h1 className="text-h1">Epic Notes Users</h1> | ||
| <h1 className="text-h1">Epic Notes Users Test</h1> |
There was a problem hiding this comment.
The change to the heading text will cause the existing end-to-end test in tests/e2e/search.test.ts to fail. The test currently checks for the exact text "Epic Notes Users" (line 11), but the heading has been updated to "Epic Notes Users Test". The test should be updated to match the new heading text.
| return ( | ||
| <div className="container mt-36 mb-48 flex flex-col items-center justify-center gap-6"> | ||
| <h1 className="text-h1">Epic Notes Users</h1> | ||
| <h1 className="text-h1">Epic Notes Users Test</h1> |
There was a problem hiding this comment.
Adding "Test" to a user-facing heading appears to be temporary for testing purposes, but should not be deployed to production. Consider using an environment variable or feature flag to control test-specific content, or ensure this change is reverted before merging to production branches.
| <h1 className="text-h1">Epic Notes Users Test</h1> | |
| <h1 className="text-h1">Epic Notes Users</h1> |
Test Plan
Checklist
Screenshots