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

Ftr 142 create detector desc text below header #150

Conversation

paladin-tech
Copy link
Contributor

@paladin-tech paladin-tech commented Nov 21, 2022

Description

Fixes description text and subtitles in Create Detector flow, as per design.

Issues Resolved

#142

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@paladin-tech paladin-tech requested a review from a team November 21, 2022 20:19
Signed-off-by: Sinisa Andric <[email protected]>
Signed-off-by: Sinisa Andric <[email protected]>
@kamingleung
Copy link

This should be related to issue #142.

Signed-off-by: Sinisa Andric <[email protected]>
@amsiglan amsiglan merged commit 17818ee into opensearch-project:main Dec 5, 2022
opensearch-trigger-bot bot pushed a commit that referenced this pull request Dec 5, 2022
* Add description text below header

Signed-off-by: Sinisa Andric <[email protected]>

* Update subtitles as per design

Signed-off-by: Sinisa Andric <[email protected]>

* Update ContentPanel subtitleText

Signed-off-by: Sinisa Andric <[email protected]>

* Fix PR remarks

Signed-off-by: Sinisa Andric <[email protected]>

* Fix subtitle size

Signed-off-by: Sinisa Andric <[email protected]>

* Cypress test fix

Signed-off-by: Sinisa Andric <[email protected]>

Signed-off-by: Sinisa Andric <[email protected]>
Co-authored-by: Amardeepsingh Siglani <[email protected]>
(cherry picked from commit 17818ee)
AWSHurneyt pushed a commit that referenced this pull request Dec 5, 2022
* Add description text below header

Signed-off-by: Sinisa Andric <[email protected]>

* Update subtitles as per design

Signed-off-by: Sinisa Andric <[email protected]>

* Update ContentPanel subtitleText

Signed-off-by: Sinisa Andric <[email protected]>

* Fix PR remarks

Signed-off-by: Sinisa Andric <[email protected]>

* Fix subtitle size

Signed-off-by: Sinisa Andric <[email protected]>

* Cypress test fix

Signed-off-by: Sinisa Andric <[email protected]>

Signed-off-by: Sinisa Andric <[email protected]>
Co-authored-by: Amardeepsingh Siglani <[email protected]>
(cherry picked from commit 17818ee)

Co-authored-by: Sinisa Andric <[email protected]>
jovancacvetkovic pushed a commit to jovancacvetkovic/security-analytics-dashboards-plugin that referenced this pull request Dec 7, 2022
* Add description text below header

Signed-off-by: Sinisa Andric <[email protected]>

* Update subtitles as per design

Signed-off-by: Sinisa Andric <[email protected]>

* Update ContentPanel subtitleText

Signed-off-by: Sinisa Andric <[email protected]>

* Fix PR remarks

Signed-off-by: Sinisa Andric <[email protected]>

* Fix subtitle size

Signed-off-by: Sinisa Andric <[email protected]>

* Cypress test fix

Signed-off-by: Sinisa Andric <[email protected]>

Signed-off-by: Sinisa Andric <[email protected]>
Co-authored-by: Amardeepsingh Siglani <[email protected]>
Signed-off-by: Jovan Cvetkovic <[email protected]>
AWSHurneyt pushed a commit to AWSHurneyt/security-analytics-dashboards-plugin that referenced this pull request Feb 22, 2023
… (opensearch-project#197)

* Add description text below header

Signed-off-by: Sinisa Andric <[email protected]>

* Update subtitles as per design

Signed-off-by: Sinisa Andric <[email protected]>

* Update ContentPanel subtitleText

Signed-off-by: Sinisa Andric <[email protected]>

* Fix PR remarks

Signed-off-by: Sinisa Andric <[email protected]>

* Fix subtitle size

Signed-off-by: Sinisa Andric <[email protected]>

* Cypress test fix

Signed-off-by: Sinisa Andric <[email protected]>

Signed-off-by: Sinisa Andric <[email protected]>
Co-authored-by: Amardeepsingh Siglani <[email protected]>
(cherry picked from commit 17818ee)

Co-authored-by: Sinisa Andric <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>
AWSHurneyt pushed a commit to AWSHurneyt/security-analytics-dashboards-plugin that referenced this pull request Oct 12, 2023
… (opensearch-project#197)

* Add description text below header

Signed-off-by: Sinisa Andric <[email protected]>

* Update subtitles as per design

Signed-off-by: Sinisa Andric <[email protected]>

* Update ContentPanel subtitleText

Signed-off-by: Sinisa Andric <[email protected]>

* Fix PR remarks

Signed-off-by: Sinisa Andric <[email protected]>

* Fix subtitle size

Signed-off-by: Sinisa Andric <[email protected]>

* Cypress test fix

Signed-off-by: Sinisa Andric <[email protected]>

Signed-off-by: Sinisa Andric <[email protected]>
Co-authored-by: Amardeepsingh Siglani <[email protected]>
(cherry picked from commit 17818ee)

Co-authored-by: Sinisa Andric <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>
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.

5 participants