Skip to content

James c fix 3270 bugs#264

Merged
James-Cocker merged 6 commits intomainfrom
JamesC-fix-3270-bugs
Feb 25, 2026
Merged

James c fix 3270 bugs#264
James-Cocker merged 6 commits intomainfrom
JamesC-fix-3270-bugs

Conversation

@James-Cocker
Copy link
Contributor

Why?

Closes galasa-dev/projectmanagement#2552 by fixed all the mentioned issues.

Changes

  • Moves the tag fetching to the testRunDetails page to avoid the lazy loading from the carbon component.
  • Makes the max-height of the table of screenshots !important to override some odd carbon behaviour (cannot test on localhost but tested this directly in browser)
  • Created cache for the api config in runsAction.ts, as well as a way to reset it for unit tests
  • Fixed 3270 loading issue by ensuring the table is rendered before flicking to the tab (unsure why this works)
  • Fixed the URL not flicking to the correct terminalImage which was introduced by the above
  • Fixed broken unit tests

Signed-off-by: James Cocker <james.s.earth@gmail.com>
Signed-off-by: James Cocker <james.s.earth@gmail.com>
Signed-off-by: James Cocker <james.s.earth@gmail.com>
Signed-off-by: James Cocker <james.s.earth@gmail.com>
Signed-off-by: James Cocker <james.s.earth@gmail.com>
Signed-off-by: James Cocker <james.s.earth@gmail.com>
@James-Cocker James-Cocker merged commit 25ed5aa into main Feb 25, 2026
8 checks passed
@James-Cocker James-Cocker deleted the JamesC-fix-3270-bugs branch February 25, 2026 14:27
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.

[BUG] 3270 Refresh Not Working Properly

2 participants