Skip to content

Remove duplicate test code - Page header components#3039

Merged
labkey-tchad merged 18 commits into
developfrom
fb_appHeaderCleanup
Jul 8, 2026
Merged

Remove duplicate test code - Page header components#3039
labkey-tchad merged 18 commits into
developfrom
fb_appHeaderCleanup

Conversation

@labkey-tchad

@labkey-tchad labkey-tchad commented Jun 9, 2026

Copy link
Copy Markdown
Member

Rationale

(Epic)

There are numerous redundant methods in various LKB test pages.
This PR targets page headers and grid menus.

Related Pull Requests

Changes

  • Use existing SampleGrid methods instead of using elements in AppBasePage.ElementCache
  • Create component classes for AppPageHeader and PageDetailHeader
  • Remove redundant manageMenu declarations
  • Add isDisplayed to base Component class

@labkey-tchad labkey-tchad self-assigned this Jun 9, 2026
@labkey-tchad labkey-tchad marked this pull request as ready for review July 6, 2026 20:09
@labkey-tchad labkey-tchad merged commit 7958301 into develop Jul 8, 2026
6 checks passed
@labkey-tchad labkey-tchad deleted the fb_appHeaderCleanup branch July 8, 2026 21:48
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.

2 participants