Skip to content

APPDUX-243: Hide config tab on OpenShift 3 clusters#598

Merged
mfrances17 merged 2 commits intointegr8ly:masterfrom
mfrances17:APPDUX-243
Jul 8, 2020
Merged

APPDUX-243: Hide config tab on OpenShift 3 clusters#598
mfrances17 merged 2 commits intointegr8ly:masterfrom
mfrances17:APPDUX-243

Conversation

@mfrances17
Copy link
Collaborator

Motivation

https://issues.redhat.com/browse/APPDUX-243

What

If running Solution Explorer on OpenShift 3, hide the Settings 'Managed Integration schedule' tab because it only applies to OpenShift 4 clusters.

Verification Steps

  1. Log onto OS4 system as an admin, click Settings, and verify that the Managed Integration schedule tab works as normal.
  2. Log onto the same OS4 system as non-admin, verify that you cannot access the same tab via the Settings cog or by adding /settings to the URL.
  3. Log onto an OS3 system as admin, click Settings, and verify that only the Solution Pattern tab is visible and works as normal.
  4. Log onto an OS3 system as admin, verify that you cannot access the single Solution Pattern content tab via the Settings cog or by adding /settings to the URL.

Checklist:

  • Code has been tested locally by PR requester
  • Changes have been successfully verified by another team member

Progress

  • Finished task

Additional Notes

Results can be verified on my OS4 system:
https://solution-explorer.apps.cluster-uxddev-916c.uxddev-916c.example.opentlc.com/

For OS3 systems, will need to be verified locally at the moment.

OS4 screen cap:
os4_settings

OS3 screen cap:
os3_settings

Copy link
Collaborator

@dlabaj dlabaj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mfrances17 mfrances17 merged commit 7e2ada6 into integr8ly:master Jul 8, 2020
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