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

Updating root readme. #195

Merged
merged 2 commits into from
Mar 19, 2021

Conversation

seanneumann
Copy link
Contributor

Signed-off-by: Neumann neumanns@amazon.com

Signed-off-by: Neumann <neumanns@amazon.com>
@mihirsoni mihirsoni requested review from mihirsoni, kavilla and boktorbb and removed request for mihirsoni and kavilla March 19, 2021 20:03
Copy link
Contributor

@mihirsoni mihirsoni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM !!

@odfe-release-bot
Copy link

✅   DCO Check Passed

Copy link
Contributor

@boktorbb boktorbb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you touch on the idea that the code currently will have bugs and issues somewhere and that it shouldn't be expected that it works perfectly? Maybe as a bullet point under the What doesn't work heading? Otherwise, looks good

README.md Outdated
* Integration test cases
* Functional test cases
* 3rd party plugins (including OpenDistro)
* Works with OpenSearch engine
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this implying it doesn't work with ElasticSearch engine?

README.md Outdated
* All references to non-Apache 2.0 code should be removed.
* Core plugins (non-3rd party)
* New project name should be used everywhere.
* yarn osd bootstrap, EsLint and commit hook should be passing.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

EsLint

nit: but should probably be ESLint

README.md Outdated
Please check to make sure someone else hasn't already created an issue for the same topic.
- Need help using OpenSearch Dashboards? Ask away on our [OpenSearch Dashboards Discuss Forum](https://discuss.elastic.co/c/kibana) and a fellow community member or
OpenSearch Dashboards engineer will be glad to help you out.
Let us know in the forums. w00t!!!
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is there a link to the forums?

@boktorbb boktorbb self-requested a review March 19, 2021 20:10

If you want to use a OpenSearch Dashboards release in production, give it a test run, or just play around:
## What doesn’t works
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: What doesn't work

Signed-off-by: Neumann <neumanns@amazon.com>
@mihirsoni mihirsoni merged commit e86f94f into opensearch-project:dashboards Mar 19, 2021
@mihirsoni mihirsoni mentioned this pull request Mar 19, 2021
kavilla referenced this pull request in kavilla/OpenSearch-Dashboards Mar 19, 2021
* Updating root readme.

Signed-off-by: Neumann <neumanns@amazon.com>

* Addressed PR comments

Signed-off-by: Neumann <neumanns@amazon.com>
kavilla pushed a commit that referenced this pull request Mar 20, 2021
* Updating root readme.

Signed-off-by: Neumann <neumanns@amazon.com>

* Addressed PR comments

Signed-off-by: Neumann <neumanns@amazon.com>
mihirsoni pushed a commit that referenced this pull request Mar 20, 2021
* Updating root readme.

Signed-off-by: Neumann <neumanns@amazon.com>

* Addressed PR comments

Signed-off-by: Neumann <neumanns@amazon.com>
wanglam added a commit to wanglam/OpenSearch-Dashboards that referenced this pull request Sep 26, 2023
…project#195)

* feat: remove management permission mode and clearify library permission mode usage

Signed-off-by: Lin Wang <wonglam@amazon.com>

* address PR comments and add annotations

Signed-off-by: Lin Wang <wonglam@amazon.com>

---------

Signed-off-by: Lin Wang <wonglam@amazon.com>
Hailong-am pushed a commit to Hailong-am/OpenSearch-Dashboards that referenced this pull request Oct 9, 2023
…project#195)

* feat: remove management permission mode and clearify library permission mode usage

Signed-off-by: Lin Wang <wonglam@amazon.com>

* address PR comments and add annotations

Signed-off-by: Lin Wang <wonglam@amazon.com>

---------

Signed-off-by: Lin Wang <wonglam@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants