-
Notifications
You must be signed in to change notification settings - Fork 394
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
DOC-875: Creds overhaul for O-R #2144
Conversation
…e old gifs and screenshots
…te, remove old png/gif
…emplate, remove old gifs
✅ Deploy Preview for n8n-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Overall readability score: 42.33 (🔴 -0.46)
View detailed metrics🟢 - Shows an increase in readability
Averages:
View metric targets
|
@StarfallProjects I think this one is now ready for review. One thing of note: I did update the Paddle cred, but it also looked to me like they've made some major updates to their API (no longer requiring a vendor ID, among other things). Paddle calls the two APIs different things (Paddle Classic is the old; Paddle Billing is the new). I noted this change in my tracking spreadsheet but I wanted to call it out here so you could review how I handled it in the creds node and see if you wanted any changes made. |
@freakwriter at the moment we only support Paddle Classic, it is a nice reminder that we should probably update the node to support the new Paddle api as well. |
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.
I think we're basically there, but had a couple of questions to clarify. See inline comments.
Creds overhaul for all credentials beginning with the letters O, P, Q, and R except for: