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

blobApi deprecations: docs #51

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ddl-giuliocapolino
Copy link

Adding deprecation message to the file content getter in Legacy APIs

apiary.apib Outdated
@@ -143,6 +143,8 @@ These endpoints allow you to interact with your project's runs and files.

### Get the contents of a file [GET]

**This endpoint is deprecated and will soon be deactivated. For more information, see the [current endpoint documentation](https://docs.dominodatalab.com/en/latest/api_guide/8c929e/rest-api-reference/#_projects).**
Copy link
Author

Choose a reason for hiding this comment

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

need to have the exact link once it is published in the documentation repo

@bianca-domino
Copy link

Hi @ddl-giuliocapolino, I'm not sure how the api-docs repo works and who should review this. For any changes to documentation in the cerebrotech/documentation repo, we have to create a Jira ticket and include that in the title of the PR in order to preview the rendered version on Gatsby. I've created a ticket for you in case you need one: https://dominodatalab.atlassian.net/browse/DOCS-2862

The text looks good to me though and I'll be happy to approve. However, to be on the safe side, can you please tag someone who works with the API docs to review this PR too?

@ddl-giuliocapolino
Copy link
Author

synced with Peter on the docs Slack channel!

apiary.apib Outdated Show resolved Hide resolved
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.

2 participants