Skip to content

Conversation

@legrego
Copy link
Member

@legrego legrego commented Jan 11, 2020

Backports the following commits to 7.x:

* Initial role mappings UI

* apply design edits

* address PR feedback

* fix type cast for number field

* Update x-pack/legacy/plugins/security/public/views/management/role_mappings/edit_role_mapping/components/mapping_info_panel/mapping_info_panel.tsx

Co-Authored-By: Joe Portner <[email protected]>

* Cleanup FTR configuration, and handle role mapping 404 errors properly

* align naming of role mappings feature check

* Apply suggestions from code review

Co-Authored-By: Brandon Kobel <[email protected]>

* add missing test assertions

* inlining feature check logic

* switch to using snapshot

* use href instead of onClick

* adding delete unit test

* consolidate href building

* unify page load error handling

* simplify initial loading state

* documenting unconditional catch blocks

* use nodes.info instead of transport.request

* Apply suggestions from code review

Co-Authored-By: Brandon Kobel <[email protected]>

* move model out of LP into NP

* convert except_field_rule to except_any_rule

* docs, take 1

* update gif

Co-authored-by: Joe Portner <[email protected]>
Co-authored-by: Brandon Kobel <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
@legrego legrego added the backport This PR is a backport of another PR label Jan 11, 2020
@legrego
Copy link
Member Author

legrego commented Jan 11, 2020

@elasticmachine merge upstream

@legrego
Copy link
Member Author

legrego commented Jan 11, 2020

Retest

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@legrego legrego merged commit 8b0b09a into elastic:7.x Jan 12, 2020
@legrego legrego deleted the backport/7.x/pr-53620 branch January 12, 2020 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants