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

MLops docs: remove 'insight key' refs and replace w/ license key #5720

Merged
merged 1 commit into from Jan 20, 2022

Conversation

zuluecho9
Copy link
Contributor

Most of work: editing MLOps docs to use 'license key' instead of 'insert key' or 'Insight key' or just 'API key'.

(Also small fix of adjusting some synthetic monitor docs code formatting.)

@zuluecho9 zuluecho9 added content requests related to docs site content from_tw Identifies issues/PRs from Tech Docs writers labels Jan 19, 2022
@zuluecho9 zuluecho9 self-assigned this Jan 19, 2022
@github-actions
Copy link

Hi @zuluecho9 👋
Thanks for your pull request! Your PR is in a queue, and a writer will take a look soon. We generally publish small edits within one business day, and larger edits within three days.

Gatsby Cloud will automatically generate a preview of your request, and will comment with a link when the preview is ready (usually 20 to 30 minutes).

@github-actions github-actions bot added this to Hero to triage in Docs PRs and Issues Jan 19, 2022
@zuluecho9 zuluecho9 requested a review from iazam6 January 19, 2022 17:45
@zuluecho9
Copy link
Contributor Author

@iazam6 One question I had was in this doc https://docs.newrelic.com/docs/mlops/integrations/datarobot-mlops-integration#connect, which references 'user key'. Should that be 'license key' also?

@zuluecho9 zuluecho9 moved this from Hero to triage to In progress/being reviewed (by Hero or any TW) in Docs PRs and Issues Jan 19, 2022
@gatsby-cloud
Copy link

gatsby-cloud bot commented Jan 19, 2022

Gatsby Cloud Build Report

docs-website-develop

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 22m

Copy link
Contributor

@urbiz-nr urbiz-nr left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this, @zuluecho9!

@zuluecho9 zuluecho9 merged commit e14d9bb into develop Jan 20, 2022
@zuluecho9 zuluecho9 deleted the Couple-small-changes---MLOps,-Synth-API-tests branch January 20, 2022 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content requests related to docs site content from_tw Identifies issues/PRs from Tech Docs writers
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants