Skip to content

Supporting changes for download tab on Cloud#36399

Merged
mcbattirola merged 14 commits intomasterfrom
mcbattirola/download-tab-on-cloud
Jan 17, 2024
Merged

Supporting changes for download tab on Cloud#36399
mcbattirola merged 14 commits intomasterfrom
mcbattirola/download-tab-on-cloud

Conversation

@mcbattirola
Copy link
Copy Markdown
Contributor

This PR includes two changes that support the issue https://github.com/gravitational/cloud/issues/6667:

  • Add a new feature flag hasSupportPageLinkAccess, since the support link was shown using the flag hasDownloadCenterListAccess, which is being modified, and
  • Update hasDownloadCenterListAccess to always return true on cloud cluster.

Contributes to https://github.com/gravitational/cloud/issues/6667

@mcbattirola mcbattirola marked this pull request as ready for review January 8, 2024 16:07
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 8, 2024

The PR changelog entry failed validation: Changelog entry not found in the PR body. Please add a "no-changelog" label to the PR, or changelog lines starting with changelog: followed by the changelog entries for the PR.

@github-actions github-actions Bot requested review from klizhentas and r0mant January 8, 2024 16:07
@mcbattirola mcbattirola added the no-changelog Indicates that a PR does not require a changelog entry label Jan 8, 2024
Comment thread web/packages/teleport/src/stores/storeUserContext.ts
}

// hasSupportPageLinkAccess checks if the user
// has access to a Support external link in the side menu.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@avatus is in the middle of removing the side bar, was the dashboard view discussed with this refactor?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This change is now in master btw: #36368

You may need to update how this is displayed @mcbattirola

Copy link
Copy Markdown
Contributor Author

@mcbattirola mcbattirola Jan 16, 2024

Choose a reason for hiding this comment

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

Thanks for pointing that out, I wasn't aware of the new topbar.

I've talked with Kenny, we'll be changing some stuff around:

Since this PR doesn't introduce the top bar link, only changes the show condition, I think we can merge this and implement the changes listed above in a new PR.

@mcbattirola mcbattirola requested a review from avatus January 9, 2024 12:42
@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from klizhentas January 17, 2024 17:13
@mcbattirola mcbattirola enabled auto-merge January 17, 2024 17:16
@mcbattirola mcbattirola added this pull request to the merge queue Jan 17, 2024
Merged via the queue into master with commit a10685e Jan 17, 2024
@mcbattirola mcbattirola deleted the mcbattirola/download-tab-on-cloud branch January 17, 2024 17:56
@public-teleport-github-review-bot
Copy link
Copy Markdown

@mcbattirola See the table below for backport results.

Branch Result
branch/v15 Create PR

mcbattirola added a commit that referenced this pull request Jan 26, 2024
* Add new flag to show/hide support link

* Lint

* Allow any user to list releases on cloud
github-merge-queue Bot pushed a commit that referenced this pull request Jan 29, 2024
* Add new flag to show/hide support link

* Lint

* Allow any user to list releases on cloud
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Indicates that a PR does not require a changelog entry size/sm ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants