[Linking omic units] Action Menu in Analysis View Update

[fatimarabab]

Pull Request Template

Checklist before requesting a review

• I have performed a self-review of my code.
• My code follows the style guidelines enforced by static analysis tools.
• If it is a core feature, I have added thorough tests.
• My changes generate no new warnings.
• New and existing unit tests pass locally with my changes.
• New or existing JSON is formatted using JQ
• I have updated the CHANGELOG to note of any product updates that will be going in with this pull request.

Pull Request Details

Wrike ticket - Refactor Analysis View Menu to incorporate Linking
Changes made:

• Updated the Action Menu in Analysis View to include menu items - Link Unit, Logout
• Updated tests.

To Review:

• Static Analysis by Reviewer

• The changes made to update the analysis view action menu are working as rendered correctly.
  To check this run the following commands:

  < Recommended command (s) provided here, modify as needed. >
  ....
  docker compose down
  docker system prune -a --volumes
  docker compose up --build -d

  • Visit https://local.rosalution.cgds/rosalution/login. Login as developer.
  • Click on any case and navigate to its Analysis View.
  • Check if the Action Menu on the right of the header renders as shown below.

• All tests pass.

• All Github Actions checks have passed.

Screenshots (if appropriate)

[SeriousHorncat]

Menu rendering!

[SeriousHorncat]

PR Looks great. There is only one thing before pulling this into main. Since it is including the linkOmicUnitOption to be included in main but the feature isn't available yet, could one of the following be done?

1. In the action menu, disable the option making it unavailable? (this can be hard-coded and removed when the feature is complete)
2. remove the option from the action menu; since the PR demonstrated it can be added with just a single line, that is sufficient enough for me that it works!

I like the idea of the first one, since it will serve as a preview of the upcoming feature if we have to go ahead and release sooner then we planned.

[fatimarabab]

PR Looks great. There is only one thing before pulling this into main. Since it is including the linkOmicUnitOption to be included in main but the feature isn't available yet, could one of the following be done?

1. In the action menu, disable the option making it unavailable? (this can be hard-coded and removed when the feature is complete)
2. remove the option from the action menu; since the PR demonstrated it can be added with just a single line, that is sufficient enough for me that it works!

I like the idea of the first one, since it will serve as a preview of the upcoming feature if we have to go ahead and release sooner then we planned.

Discussed in stand up to keep this branch as a feature branch for Linking 'omic units, and merge it as a completed feature into main.