Update heading text on users page to include "Test" for verification …#27
Update heading text on users page to include "Test" for verification …#27kasperpeulen wants to merge 1 commit intomainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR modifies the heading text on the users page by appending "Test" to the existing "Epic Notes Users" heading, apparently for verification purposes as indicated in the PR title.
- Changes the h1 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.
Adding "Test" to the heading text creates a mismatch with the existing e2e test in tests/e2e/search.test.ts at line 11, which expects the exact text "Epic Notes Users". This will cause the test to fail. Additionally, this change appears to be for temporary verification purposes based on the PR title, but adding test-related text to production UI is not a recommended practice. If this is truly needed for verification, consider updating the corresponding test or using a different verification approach that doesn't modify user-facing text.
| <h1 className="text-h1">Epic Notes Users Test</h1> | |
| <h1 className="text-h1">Epic Notes Users</h1> |
1 participant
…purposes
Test Plan
Checklist
Screenshots