n8n-docs icon indicating copy to clipboard operation
n8n-docs copied to clipboard

Docs: Add Airtop’s documentation

Open cesars-gh opened this issue 8 months ago • 3 comments

About

Adding node and credential documentation for Airtop’s built-in integration.

cesars-gh avatar Mar 10 '25 19:03 cesars-gh

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Mar 10 '25 19:03 CLAassistant

Deploy Preview for n8n-docs ready!

Name Link
Latest commit 7cfa5dae64442a9a4352d47d325a58b7714f2666
Latest deploy log https://app.netlify.com/sites/n8n-docs/deploys/67ed0fda063b5f0008f237fe
Deploy Preview https://deploy-preview-2952--n8n-docs.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Mar 10 '25 19:03 netlify[bot]

Thanks for the PR @cesars-gh! We'll keep an eye on the PR for the node and credential since that'll have to be merged in first. Once that's done, we will take a look at the docs pages and let you know if any changes are needed.

Thanks again!

imchairmanm avatar Mar 11 '25 13:03 imchairmanm

Looks like the PR for the main repo was merged, so this is unblocked now.

imchairmanm avatar Apr 02 '25 08:04 imchairmanm

@cesars-gh, would you mind allowing us edit permissions on your fork? It'd make it easier for us to resolve conflicts and make any changes to the PR.

imchairmanm avatar Apr 02 '25 08:04 imchairmanm

@cesars-gh, would you mind allowing us edit permissions on your fork? It'd make it easier for us to resolve conflicts and make any changes to the PR.

Hi @imchairmanm, sure, I'll give you access to our fork. Should we add someone else besides you?

cesars-gh avatar Apr 02 '25 09:04 cesars-gh

@cesars-gh, would you mind allowing us edit permissions on your fork? It'd make it easier for us to resolve conflicts and make any changes to the PR.

Hi @imchairmanm, sure, I'll give you access to our fork. Should we add someone else besides you?

Thanks @cesars-gh! @evilnick would be the other person who might review this.

imchairmanm avatar Apr 02 '25 09:04 imchairmanm

Hey @cesars-gh, this looks good! I pushed a few small changes and merged in changes from main. I think this is good to go once the Airtop node and cred are included in a proper release. I'll hold off on approval until then to avoid confusion, but I think this is good to go. Thanks for your work on this.

imchairmanm avatar Apr 02 '25 10:04 imchairmanm

Hi @imchairmanm, first of all, thank you for reviewing and improving this documentation. I wanted to ask you if you know how can we link Airtop templates to the node's documentation here: image The link to Airtop integrations throws 404. How can I add it?

cesars-gh avatar Apr 08 '25 13:04 cesars-gh

@cesars-gh thanks for the heads up. The integrations page is usually created async by our marketing team, but I can try to ping them to ask about it. I know it's not great to have that link fail during the interim.

imchairmanm avatar Apr 08 '25 15:04 imchairmanm

@cesars-gh thanks for the heads up. The integrations page is usually created async by our marketing team, but I can try to ping them to ask about it. I know it's not great to have that link fail during the interim.

I'd appreciate it if you talk to them 🙏 Thank you!

cesars-gh avatar Apr 08 '25 15:04 cesars-gh

Hey @cesars-gh, so I checked in with the team and there have been a few delays with getting this live since we're still trying to fix some rough edges from our recent website redesign. The team is planning on doing a deploy this afternoon though that should include the Airtop integration page, so :crossed_fingers: that it's fixed this afternoon.

imchairmanm avatar Apr 09 '25 08:04 imchairmanm

Looks like the page is live now: https://n8n.io/integrations/airtop/

imchairmanm avatar Apr 09 '25 09:04 imchairmanm

Looks like the page is live now: https://n8n.io/integrations/airtop/

Thank you very much for your help @imchairmanm 🎩

cesars-gh avatar Apr 09 '25 20:04 cesars-gh