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

Update NotificationEventIndexTest.kt #1

Open
wants to merge 33 commits into
base: test-review
Choose a base branch
from

Conversation

kavithacm
Copy link
Owner

Description

[Describe what this change achieves]

Issues Resolved

[List any issues this PR will resolve]

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test and doctest
  • New functionality has been documented.
    • New functionality has javadoc added
    • New functionality has user manual doc added
  • 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.

akbhatta and others added 30 commits October 6, 2020 11:00
TODO:
Need to create header links for easy navigation
* Add docs to main branch

* Update pull request template

Signed-off-by: Joshua Li <[email protected]>

* Addrss comments

Signed-off-by: Joshua Li <[email protected]>

* Update readme from develop branch

Signed-off-by: Joshua Li <[email protected]>
…arch-project#250)

* add tooltip_support to spi settings

* add host deny list in setting and apply to url validator
* Fix custom webhook request body format

* support http protocol for webhook url
[Tests]
All unit and integration test passed

Signed-off-by: @akbhatta
* exclude test to pass CI

* bump version for notification-opensearch to 1.1
resolved conflicts and refactored sns

[Tests]
Updated tests for the modified interface

Signed-off-by: @akbhatta
[Tests]
Updated tests to use String

Signed-off-by: @akbhatta
…rch-project#254)

* Adds PublishNotificationAction for legacy ISM implementation

Signed-off-by: Drew Baugher <[email protected]>

* Fixes rename of feature and required referenceId from merged changes

Signed-off-by: Drew Baugher <[email protected]>
[Tests]
Added intergration tests for config operations

Signed-off-by: @akbhatta
* add backend metrics for notifications
Signed-off-by: David Cui <[email protected]>

* add test cases for counters
Signed-off-by: David Cui <[email protected]>

* addressed comments, refactored send message usage after merging from upstream
Signed-off-by: David Cui <[email protected]>

* add channel metrics from deprecated front-end metrics collection
Signed-off-by: David Cui <[email protected]>

* removed redundant metrics
Signed-off-by: David Cui <[email protected]>

* refactored Metrics to kotlin, grouped compile statement in build.gradle
Signed-off-by: David Cui <[email protected]>
…opensearch-project#295)

Bumps [hosted-git-info](https://github.com/npm/hosted-git-info) from 2.8.8 to 2.8.9.
- [Release notes](https://github.com/npm/hosted-git-info/releases)
- [Changelog](https://github.com/npm/hosted-git-info/blob/v2.8.9/CHANGELOG.md)
- [Commits](npm/hosted-git-info@v2.8.8...v2.8.9)

---
updated-dependencies:
- dependency-name: hosted-git-info
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…nsearch-project#294)

Bumps [path-parse](https://github.com/jbgutierrez/path-parse) from 1.0.6 to 1.0.7.
- [Release notes](https://github.com/jbgutierrez/path-parse/releases)
- [Commits](https://github.com/jbgutierrez/path-parse/commits/v1.0.7)

---
updated-dependencies:
- dependency-name: path-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…project#293)

Bumps [ws](https://github.com/websockets/ws) from 7.4.4 to 7.5.3.
- [Release notes](https://github.com/websockets/ws/releases)
- [Commits](websockets/ws@7.4.4...7.5.3)

---
updated-dependencies:
- dependency-name: ws
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…pensearch-project#292)

Bumps [browserslist](https://github.com/browserslist/browserslist) from 4.16.3 to 4.16.8.
- [Release notes](https://github.com/browserslist/browserslist/releases)
- [Changelog](https://github.com/browserslist/browserslist/blob/main/CHANGELOG.md)
- [Commits](browserslist/browserslist@4.16.3...4.16.8)

---
updated-dependencies:
- dependency-name: browserslist
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
kavithacm pushed a commit that referenced this pull request Nov 8, 2021
* Migrate frontend to Opensearch

* Add readme for dashboards notifications

* Move notifications-kibana-plugin to dashboards-notifications

* Remove opendistro in dashboards notifications

* Update documents for repo

* rename kibana

* Address comments
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.

10 participants