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] Reverting notebooks changes, add docs/validation to datasources, explorer minor fixes #1102

Merged
merged 1 commit into from
Oct 9, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 4a8b500 from #1101.

…orer minor fixes (#1101)

* Add documentation linking and validation for query permissions

Signed-off-by: Derek Ho <dxho@amazon.com>

* Update test with link

Signed-off-by: Derek Ho <dxho@amazon.com>

* Add form validation for name field

Signed-off-by: Derek Ho <dxho@amazon.com>

* reverting notebook changes from #974 & #985

Signed-off-by: Shenoy Pratik <sgguruda@amazon.com>

* Remove console log

Signed-off-by: Derek Ho <dxho@amazon.com>

* surrounding events functionally working

Signed-off-by: Paul Sebastian <paulstn@amazon.com>

* small surrounding flyout ui changes

Signed-off-by: Paul Sebastian <paulstn@amazon.com>

---------

Signed-off-by: Derek Ho <dxho@amazon.com>
Signed-off-by: Shenoy Pratik <sgguruda@amazon.com>
Signed-off-by: Paul Sebastian <paulstn@amazon.com>
Co-authored-by: Shenoy Pratik <sgguruda@amazon.com>
Co-authored-by: Paul Sebastian <paulstn@amazon.com>
(cherry picked from commit 4a8b500)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Oct 9, 2023

Codecov Report

Merging #1102 (ad1f735) into 2.x (c9d5367) will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              2.x    #1102      +/-   ##
==========================================
+ Coverage   44.09%   44.23%   +0.13%     
==========================================
  Files         325      327       +2     
  Lines       19378    19478     +100     
  Branches     4617     4643      +26     
==========================================
+ Hits         8545     8616      +71     
- Misses      10788    10817      +29     
  Partials       45       45              
Flag Coverage Δ
dashboards-observability 44.23% <ø> (+0.13%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 10 files with indirect coverage changes

@ps48 ps48 merged commit 2d4afc3 into 2.x Oct 9, 2023
11 checks passed
@github-actions github-actions bot deleted the backport/backport-1101-to-2.x branch October 9, 2023 23:29
amsiglan pushed a commit to amsiglan/dashboards-observability that referenced this pull request Jun 7, 2024
…orer minor fixes (opensearch-project#1101) (opensearch-project#1102)

* Add documentation linking and validation for query permissions

* Update test with link

* Add form validation for name field

* reverting notebook changes from opensearch-project#974 & opensearch-project#985

* Remove console log

* surrounding events functionally working

* small surrounding flyout ui changes

---------

(cherry picked from commit 4a8b500)

Signed-off-by: Derek Ho <dxho@amazon.com>
Signed-off-by: Shenoy Pratik <sgguruda@amazon.com>
Signed-off-by: Paul Sebastian <paulstn@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Shenoy Pratik <sgguruda@amazon.com>
Co-authored-by: Paul Sebastian <paulstn@amazon.com>
(cherry picked from commit 2d4afc3)
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