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

[PURIFY][DOCS] Regenerate API docs after rename #250

Merged
merged 1 commit into from
Apr 6, 2021

Conversation

mihirsoni
Copy link
Contributor

@mihirsoni mihirsoni commented Apr 5, 2021

Signed-off-by: Mihir Soni mihirsoni.123@gmail.com

Resolves -> #249

Signed-off-by: Mihir Soni <mihirsoni.123@gmail.com>
@mihirsoni mihirsoni added docs Improvements or additions to documentation >Fork labels Apr 5, 2021
@odfe-release-bot
Copy link

✅   DCO Check Passed a40f8e1

Copy link
Member

@kavilla kavilla left a comment

Choose a reason for hiding this comment

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

LGTM, it caught an accidental update.

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.

LGTM just a couple of questions

@@ -894,7 +824,7 @@ export type IBasePath = Pick<BasePath, keyof BasePath>;

// @public
export interface IClusterClient {
readonly asInternalUser: ElasticsearchClient;
readonly asInternalUser: OpenSearchClient;
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this stay as ElasticsearchClient?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think it should OpenSearch as it is internal.

Comment on lines +936 to +937
readonly asCurrentUser: OpenSearchClient;
readonly asInternalUser: OpenSearchClient;
Copy link
Contributor

Choose a reason for hiding this comment

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

Should these stay as ElasticsearchClient?

@boktorbb boktorbb self-requested a review April 6, 2021 00:11
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.

LGTM

@mihirsoni mihirsoni merged commit 3ee5476 into opensearch-project:main Apr 6, 2021
wanglam added a commit to wanglam/OpenSearch-Dashboards that referenced this pull request Feb 26, 2024
SuZhou-Joe pushed a commit to SuZhou-Joe/OpenSearch-Dashboards that referenced this pull request Mar 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants