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.

Sign up for GitHub

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

Jump to bottom

[DOCS] Update doc URLs for trained model deployment APIs #75388

Merged
merged 1 commit into from
Jul 15, 2021
Merged

[DOCS] Update doc URLs for trained model deployment APIs #75388

merged 1 commit into from
Jul 15, 2021

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Jul 15, 2021

Related to #73660

This PR updates the documentation URL in some API specs to address the following errors:

09:08:23 INFO:build_docs:Bad cross-document links:
09:08:23 INFO:build_docs: /tmp/docsbuild/target_repo/html/en/elasticsearch/client/javascript-api/master/api-reference.html contains broken links to:
09:08:23 INFO:build_docs: - en/elasticsearch/reference/master/ml-infer-trained-model-deployment.html
09:08:23 INFO:build_docs: - en/elasticsearch/reference/master/ml-start-trained-model-deployment.html
09:08:23 INFO:build_docs: - en/elasticsearch/reference/master/stop-trained-model-deployment.html

Once the documentation for those APIs exists, we can update the URLs againi.

@lcawl lcawl added >docs General docs changes :ml Machine learning v8.0.0 labels Jul 15, 2021
@elasticmachine elasticmachine added Team:ML Meta label for the ML team Team:Docs Meta label for docs team labels Jul 15, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (Team:ML)

@sethmlarson sethmlarson added the Team:Clients Meta label for clients team label Jul 15, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/clients-team (Team:Clients)

sethmlarson
sethmlarson approved these changes Jul 15, 2021
View reviewed changes
Copy link
Contributor

@sethmlarson sethmlarson left a comment

Choose a reason for hiding this comment

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

LGTM, is there an issue tracking these API documentation pages getting created? If so should add updating these URLs again to the list if not already.

@lcawl lcawl merged commit 345c00c into elastic:master Jul 15, 2021
@lcawl lcawl deleted the trained-model-specs branch July 15, 2021 17:51
masseyke pushed a commit to masseyke/elasticsearch that referenced this pull request Jul 16, 2021
@lcawl @masseyke
[DOCS] Update doc URLs for trained model deployment APIs (elastic#75388)
6d18e14
ywangd pushed a commit to ywangd/elasticsearch that referenced this pull request Jul 30, 2021
@lcawl @ywangd
[DOCS] Update doc URLs for trained model deployment APIs (elastic#75388)
b2a317f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sethmlarson sethmlarson sethmlarson approved these changes

Assignees
No one assigned
Labels
>docs General docs changes :ml Machine learning Team:Clients Meta label for clients team Team:Docs Meta label for docs team Team:ML Meta label for the ML team v8.0.0-alpha1
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@lcawl @elasticmachine @sethmlarson @jakelandis