Skip to content

[Backport 2.x] Reverting notebooks changes, add docs/validation to datasources, explorer minor fixes#1102

Merged
ps48 merged 1 commit into2.xfrom
backport/backport-1101-to-2.x
Oct 9, 2023
Merged

[Backport 2.x] Reverting notebooks changes, add docs/validation to datasources, explorer minor fixes#1102
ps48 merged 1 commit into2.xfrom
backport/backport-1101-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
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
Copy Markdown

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
@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