Skip to content

Conversation

@TCMeldrum
Copy link
Contributor

@TCMeldrum TCMeldrum commented Nov 17, 2025

Closes: UserOfficeProject/issue-tracker#1495

Description

This PR enhances the clarity of the descriptions in the tech review boxes, making them more detailed and understandable for users.

Motivation and Context

The change was necessary to provide more detailed instructions to the reviewers, specifically informing them about the visibility and purpose of their comments and uploaded documents. This improvement aims to prevent confusion and miscommunication during the review process.

Changes

  1. Adjusted the descriptions for "Internal comment" and "Internal documents" fields to indicate that their content is visible only to other technical reviewers.
  2. Altered the description for "Comments for the review panel" field to specify its visibility to the FAP panels.
  3. Implemented the use of 'useTranslation' function for better internationalization support.

How Has This Been Tested?

Fixes Jira Issue

https://jira.esss.lu.se/browse/

Depends On

Tests included/Docs Updated?

  • I have added tests to cover my changes.
  • All relevant doc has been updated

@gnyiri gnyiri self-requested a review November 18, 2025 14:56
@github-actions github-actions bot force-pushed the 1495-tech-review-box-description branch from deacba9 to 1f6844b Compare November 21, 2025 10:47
@TCMeldrum TCMeldrum marked this pull request as ready for review November 21, 2025 10:48
@TCMeldrum TCMeldrum requested a review from a team as a code owner November 21, 2025 10:48
@TCMeldrum TCMeldrum requested review from mehta-pooja123 and removed request for a team November 21, 2025 10:48
@github-actions github-actions bot force-pushed the 1495-tech-review-box-description branch from 1f6844b to 3098d69 Compare November 25, 2025 16:22
@TCMeldrum
Copy link
Contributor Author

@gnyiri You have self requested a review, are you happy with the changes?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make tech review box titles clearer

5 participants