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

Contentful integration (Delivery/Preview API) #739

Merged
merged 4 commits into from
Aug 25, 2020

Conversation

0xsven
Copy link
Contributor

@0xsven 0xsven commented Jul 9, 2020

Reference: https://www.contentful.com/developers/docs/references/content-delivery-api/#/reference

Spaces:

  • Get current

Content Types:

  • Get all
  • Get single

Entries:

  • Get all
  • Get single

Assets:

  • Get all
  • Get single

Locales:

  • Get all

Search Parameters:

  • add as many as you like

This node only supports Contentful's Delivery and Preview APIs. I would like to add the management API in the future, which needs different credentials. Would you create a second node or add it two this node?

@@ -0,0 +1,156 @@
// @ts-nocheck
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This code is part of the official resolve package. Not sure how to integrate it. I didn't want to add a dependency for just one node.

type: 'string' as NodePropertyTypes,
default: '',
required: true,
description: 'The id for the Cotentful space.'
Copy link

Choose a reason for hiding this comment

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

The id for the Cotentful space. 👉 The id for the Contentful space.

The n is missing

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍

@janober janober force-pushed the master branch 2 times, most recently from e4cdc72 to 941ee06 Compare July 12, 2020 10:17
@janober janober merged commit 6573642 into n8n-io:master Aug 25, 2020
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.

3 participants