Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[UII] Remove action links and buttons when secrets field is disabled #188252

Merged
merged 3 commits into from
Jul 12, 2024

Conversation

jen-huang
Copy link
Contributor

@jen-huang jen-huang commented Jul 12, 2024

Summary

Resolves #187642

This PR removes action links and buttons to replace and convert secrets fields if the field itself is disabled, for cases like the user not having write permissions:

image image

Also applies sentence casing to Service token label.

Checklist

@jen-huang jen-huang added release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Jul 12, 2024
@jen-huang jen-huang self-assigned this Jul 12, 2024
@jen-huang jen-huang requested a review from a team as a code owner July 12, 2024 16:39
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@obltmachine
Copy link

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@jen-huang jen-huang changed the title [UII] Capitalization fix [UII] Remove action links and buttons when secrets field is disabled Jul 12, 2024
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
fleet 1.8MB 1.8MB +271.0B

cc @jen-huang

@jen-huang jen-huang enabled auto-merge (squash) July 12, 2024 18:17
Copy link
Member

@nchaulet nchaulet left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@jen-huang jen-huang merged commit 4c0db61 into elastic:main Jul 12, 2024
29 checks passed
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 12, 2024
…lastic#188252)

## Summary

Resolves elastic#187642

This PR removes action links and buttons to replace and convert secrets
fields if the field itself is disabled, for cases like the user not
having write permissions:

<img width="730" alt="image"
src="https://github.com/user-attachments/assets/957ad858-74bf-4f28-a212-148253e4ca0f">

<img width="716" alt="image"
src="https://github.com/user-attachments/assets/d70f7946-8b62-4e13-9d16-98ddac26b826">

Also applies sentence casing to `Service token` label.

### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

(cherry picked from commit 4c0db61)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.15

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

@jen-huang jen-huang deleted the fix/187642 branch July 12, 2024 19:01
kibanamachine added a commit that referenced this pull request Jul 12, 2024
…sabled (#188252) (#188258)

# Backport

This will backport the following commits from `main` to `8.15`:
- [[UII] Remove action links and buttons when secrets field is disabled
(#188252)](#188252)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Jen
Huang","email":"its.jenetic@gmail.com"},"sourceCommit":{"committedDate":"2024-07-12T18:31:16Z","message":"[UII]
Remove action links and buttons when secrets field is disabled
(#188252)\n\n## Summary\r\n\r\nResolves #187642 \r\n\r\nThis PR removes
action links and buttons to replace and convert secrets\r\nfields if the
field itself is disabled, for cases like the user not\r\nhaving write
permissions:\r\n\r\n<img width=\"730\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/957ad858-74bf-4f28-a212-148253e4ca0f\">\r\n\r\n<img
width=\"716\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/d70f7946-8b62-4e13-9d16-98ddac26b826\">\r\n\r\nAlso
applies sentence casing to `Service token` label.\r\n\r\n###
Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"4c0db61e6d92ada57833471a33daa5eb40c38e44","branchLabelMapping":{"^v8.16.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Fleet","backport:prev-minor","v8.16.0"],"title":"[UII]
Remove action links and buttons when secrets field is
disabled","number":188252,"url":"https://github.com/elastic/kibana/pull/188252","mergeCommit":{"message":"[UII]
Remove action links and buttons when secrets field is disabled
(#188252)\n\n## Summary\r\n\r\nResolves #187642 \r\n\r\nThis PR removes
action links and buttons to replace and convert secrets\r\nfields if the
field itself is disabled, for cases like the user not\r\nhaving write
permissions:\r\n\r\n<img width=\"730\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/957ad858-74bf-4f28-a212-148253e4ca0f\">\r\n\r\n<img
width=\"716\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/d70f7946-8b62-4e13-9d16-98ddac26b826\">\r\n\r\nAlso
applies sentence casing to `Service token` label.\r\n\r\n###
Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"4c0db61e6d92ada57833471a33daa5eb40c38e44"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/188252","number":188252,"mergeCommit":{"message":"[UII]
Remove action links and buttons when secrets field is disabled
(#188252)\n\n## Summary\r\n\r\nResolves #187642 \r\n\r\nThis PR removes
action links and buttons to replace and convert secrets\r\nfields if the
field itself is disabled, for cases like the user not\r\nhaving write
permissions:\r\n\r\n<img width=\"730\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/957ad858-74bf-4f28-a212-148253e4ca0f\">\r\n\r\n<img
width=\"716\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/d70f7946-8b62-4e13-9d16-98ddac26b826\">\r\n\r\nAlso
applies sentence casing to `Service token` label.\r\n\r\n###
Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"4c0db61e6d92ada57833471a33daa5eb40c38e44"}}]}]
BACKPORT-->

Co-authored-by: Jen Huang <its.jenetic@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v8.15.0 v8.16.0
Projects
None yet
5 participants