Skip to content

[v14] Show Connect My Computer button in empty state in Connect#33440

Merged
ravicious merged 1 commit intobranch/v14from
ravicious/v14/backport-33144-empty-state
Oct 13, 2023
Merged

[v14] Show Connect My Computer button in empty state in Connect#33440
ravicious merged 1 commit intobranch/v14from
ravicious/v14/backport-33144-empty-state

Conversation

@ravicious
Copy link
Copy Markdown
Member

@ravicious ravicious commented Oct 13, 2023

Backport #33144.

The bot wasn't able to backport the PR cleanly because I forgot to rebase #33144 after it's base branch changed to master from #33142. #33142 was backported to v14 separately.

* Reformat buttons story away from using spacing on individual elements

* Add support for gap prop to Button

* Fix and expand DocumentCluster stories

* Add info about Discover to empty text in DocumentCluster

* hasConnectMyComputerPermissions: Accept LoggedInUser instead of Cluster

* Refactor getEmptyTableText into two separate functions

This will make it easier to show the Connect My Computer button in the
table with servers.

* Show button for Connect My Computer in empty state

* Show hint about adding resources on separate line

* Do not show resources hint in leaf clusters
@ravicious ravicious added this pull request to the merge queue Oct 13, 2023
Merged via the queue into branch/v14 with commit fdd141a Oct 13, 2023
@ravicious ravicious deleted the ravicious/v14/backport-33144-empty-state branch October 13, 2023 10:24
@camscale camscale mentioned this pull request Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants