Skip to content

chore: move public comment above internal comment - #1543

Merged
TCMeldrum merged 5 commits into
developfrom
1601-move-public-comment-above-internal
Jul 13, 2026
Merged

chore: move public comment above internal comment#1543
TCMeldrum merged 5 commits into
developfrom
1601-move-public-comment-above-internal

Conversation

@TCMeldrum

@TCMeldrum TCMeldrum commented May 27, 2026

Copy link
Copy Markdown
Contributor

Description

This PR moves the "Public Comment" section above the "Internal Comment" section in the frontend application's "Technical Review Basis" component.

Motivation and Context

This change improves the user experience by placing the "Public Comment" section, which is often more frequently used and accessed by users, in a more prominent position on the page.

Changes

  • The "Public Comment" section has been moved from below the "Internal Comment" section to above it.
  • The associated event handlers and state management logic have been updated to reflect this change.
  • No functionalities were changed, only the order of the sections has been altered.

How Has This Been Tested?

Fixes Jira Issue

https://jira.ess.eu//browse/

Depends On

Tests included/Docs Updated?

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

@TCMeldrum
TCMeldrum marked this pull request as ready for review June 22, 2026 14:05
@TCMeldrum
TCMeldrum requested a review from a team as a code owner June 22, 2026 14:05
@TCMeldrum
TCMeldrum requested review from ACLay and removed request for a team June 22, 2026 14:05
@TCMeldrum
TCMeldrum enabled auto-merge (squash) July 9, 2026 14:40
@TCMeldrum
TCMeldrum merged commit 4fdf7b8 into develop Jul 13, 2026
21 checks passed
@TCMeldrum
TCMeldrum deleted the 1601-move-public-comment-above-internal branch July 13, 2026 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants