-
Notifications
You must be signed in to change notification settings - Fork 362
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
Adds docs for new credentials #2221
Conversation
✅ Deploy Preview for n8n-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Overall readability score: 42.13 (🔴 -0.09)
View detailed metrics🟢 - Shows an increase in readability
Averages:
View metric targets
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Joffcom I added some suggested commits here, mostly to bold the GUI fields in n8n, sometimes to add some additional links to the provider's docs. I also already committed some changes to the Vale checking so it would stop throwing errors about Datadog, DFIR-IRIS and Malcore. Should be good to go if you accept the suggested commits.
Co-authored-by: Kate Mueller <freakwriter@users.noreply.github.com>
Co-authored-by: Kate Mueller <freakwriter@users.noreply.github.com>
Co-authored-by: Kate Mueller <freakwriter@users.noreply.github.com>
Co-authored-by: Kate Mueller <freakwriter@users.noreply.github.com>
Co-authored-by: Kate Mueller <freakwriter@users.noreply.github.com>
Co-authored-by: Kate Mueller <freakwriter@users.noreply.github.com>
Thanks @freakwriter |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most of the links to marketing integration pages are 404ing, but I'm assuming that's because the nodes haven't actually been released yet. Dynatrace did seem like it was out there though, so I added a comment in case that's just the wrong URL. Let me know if it's not and I can approve as-is.
I added contentType: integration into the frontmatter for all of these, since they should have it but it seemed silly to ask you to do that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Joffcom good to ship from my end!
No description provided.