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

feat(Postmark Trigger Node): Update credentials so they can be used in the HTTP Request Node #4790

Merged
merged 2 commits into from
Dec 1, 2022

Conversation

bramkn
Copy link
Contributor

@bramkn bramkn commented Dec 1, 2022

Add new way of doing credentials so they also show up in HTTP request node.
As we only have a trigger node now for postmark, this helps us a lot adding API requests through the HTTP request without the need to setup the credentials 2x.

@bramkn bramkn changed the title change Credentials to be also usable with HTTP requests feat: change Credentials to be also usable with HTTP requests Dec 1, 2022
@bramkn bramkn changed the title feat: change Credentials to be also usable with HTTP requests Feat: change Credentials to be also usable with HTTP requests Dec 1, 2022
@bramkn bramkn changed the title Feat: change Credentials to be also usable with HTTP requests feat: Change Credentials to be also usable with HTTP requests Dec 1, 2022
@Joffcom
Copy link
Member

Joffcom commented Dec 1, 2022

Hey @bramkn,

Thanks for the PR, I am just going to check this one now quickly before signing off for the day.

@n8n-assistant n8n-assistant bot added community Authored by a community member node/improvement New feature or request labels Dec 1, 2022
@Joffcom
Copy link
Member

Joffcom commented Dec 1, 2022

Hey @bramkn,

I have given the trigger node a test and used the credential in the HTTP request node and all looks to be good. Thanks for the time you spent on this PR, I will get it merged now.

@Joffcom Joffcom changed the title feat: Change Credentials to be also usable with HTTP requests feat(Postmark Trigger Node): Update credentials so they can be used in the HTTP Request Node Dec 1, 2022
@Joffcom Joffcom merged commit 0c759dc into n8n-io:master Dec 1, 2022
@Joffcom
Copy link
Member

Joffcom commented Dec 1, 2022

Thanks again @bramkn, This has now been merged and will be available in the next release.

@n8n-assistant n8n-assistant bot added the Upcoming Release Will be part of the upcoming release label Dec 1, 2022
MiloradFilipovic added a commit that referenced this pull request Dec 1, 2022
* master:
  feat(Postmark Trigger Node): Update credentials so they can be used with the HTTP Request Node (#4790)
  fix(editor): Remove unnecessary closing tag in CanvasControls (no-changelog) (#4792)
  feat(Facebook Graph API Node): Update to support api version 15 (#4791)
  fix(Pipedrive Node): Resolve properties not working
  fix(editor): Remove excess `isReadOnly` prop (no-changelog) (#4786)
  fix(core): OAuth2 scope saved to DB fix
  feat(editor): Overhaul expression editor modal (#4631)
  test(editor): Intercept credentials save and test request and wait for them in e2e test (#4778)
  fix(Google Sheets  Node): Fix for append operation if no empty rows in sheet
@janober
Copy link
Member

janober commented Dec 2, 2022

Got released with n8n@0.205.0

@janober janober removed the Upcoming Release Will be part of the upcoming release label Dec 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Authored by a community member node/improvement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants