[7.x] Btsymbala/trusted app deletion (#77316) #78299
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Moved the DeleteTrustedAppsRequestParams to common folder to be able to use it on the client. * Added trusted app deletion API to the client service layer. * Made default data type for async resource state. * Added guard for stale state. * Added timestamp to the list data to be used to refresh list when it's modified. * Separated out base type for resource state change actions. * Added action for outdating list data. * Moved the refresh condition inside the middleware case function and added timestamping data. * Added state, actions, reducers and middleware for deletion dialog. * Added actions column and deletion action. * Added trusted app deletion dialog. * Changed to not have deletonDialog as optional in store. * Changed the store to contain the full entry in the dialog state and changed the modal message to indicate the trusted app name. * Extracted notifications component and enhanced error display. * Added success message and unified messages a bit. * Complete coverage with tests. * Removed unused variable in translations. * Fixed tests because of outdated snapshots and inproper mocking of htmlIdGenerator. * Fixed code review comments. * Fixed type error.
💚 Build SucceededBuild metrics@kbn/optimizer bundle module count
async chunks size
distributable file count
To update your PR or re-run it, just comment with: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Backports the following commits to 7.x: