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

[Backport 2.x] Add DocView links pluggable injection capability #1891

Merged
merged 1 commit into from
Jul 20, 2022

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport c33bf8c from #1200

@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team as a code owner July 13, 2022 18:48
* Add DocView links pluggable injection capability
* Add proper e2e + bugfix
* set prop as lower case
* Better name
* Delete prop after usage
* Camel case + avoid mutating
* Added right data-test-subj in order to fix tests
* update snapshot
* Add hide prop in order to mimic ng-if condition
* Add i18n translation support
* Fix tests
* Made text smaller + used eui components instead of custom css
* Snapshots

Signed-off-by: sitbubu <royi.sitbon@logz.io>
Signed-off-by: Royi Sitbon <royi.sitbon@logz.io>

(cherry picked from commit c33bf8c)
@kavilla kavilla force-pushed the backport/backport-1200-to-2.x branch from ec213b7 to 8b846f3 Compare July 13, 2022 20:32
@joshuarrrr joshuarrrr merged commit 1928f4a into 2.x Jul 20, 2022
@joshuarrrr joshuarrrr deleted the backport/backport-1200-to-2.x branch July 20, 2022 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants