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

fix(Salesforce Node): Fix typo for adding a contact to a campaign #6598

Merged
merged 1 commit into from
Jul 5, 2023

Conversation

Joffcom
Copy link
Member

@Joffcom Joffcom commented Jul 5, 2023

Github issue / Community forum post (link here to close automatically): #6593

@github-actions
Copy link
Contributor

github-actions bot commented Jul 5, 2023

Great PR! Please pay attention to the following items before merging:

Files matching packages/**:

  • If fixing bug, added test to cover scenario.
  • If addressing forum or Github issue, added link to description.

Files matching packages/**/*.ts:

  • Added unit tests to cover new or updated functionality.

Files matching packages/nodes-base/nodes/**:

  • Added workflow tests for nodes if possible.

Make sure to check off this list before asking for review.

@cypress
Copy link

cypress bot commented Jul 5, 2023

Passing run #1381 ↗︎

0 228 0 0 Flakiness 0

Details:

🌳 gh/6593 🖥️ browsers:node18.12.0-chrome107 🤖 Joffcom 🗃️ e2e/*
Project: n8n Commit: 065e8621bb
Status: Passed Duration: 07:39 💡
Started: Jul 5, 2023 10:14 AM Ended: Jul 5, 2023 10:21 AM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 5, 2023

✅ All Cypress E2E specs passed

@n8n-assistant n8n-assistant bot added n8n team Authored by the n8n team node/improvement New feature or request labels Jul 5, 2023
@Joffcom Joffcom merged commit 7ffe3cb into master Jul 5, 2023
@Joffcom Joffcom deleted the gh/6593 branch July 5, 2023 10:37
@janober
Copy link
Member

janober commented Jul 5, 2023

Got released with [email protected]

MiloradFilipovic added a commit that referenced this pull request Jul 6, 2023
* master: (50 commits)
  fix(Google Drive Node): Fix regex in file RLC (#6607)
  feat(HTML Node): 'Convert to table operation (#6540)
  ci: e2e tests don't need to enable saving manual executions
  🚀 Release 1.0.1 (#6606)
  fix(core): Make node execution order configurable, and backward-compatible (#6507)
  🚀 Release 0.236.0 (#6602)
  fix: Fix migrations for postgres and mysql (no-changelog) (#6600)
  fix(core): Improve the performance of last 2 sqlite migrations (#6522)
  refactor: Switch cloud plan retrieval endpoint (no-changelog) (#6529)
  fix(Salesforce Node): Fix typo for adding a contact to a campaign (#6598)
  ci: Hide v1 banner in e2e tests to fix 2-credentials.cy.ts (no-changelog)
  fix(editor): Restore expression completions (#6566)
  refactor: Remove basic auth env vars from docker compose stacks (#6570)
  fix(core): Fix migrations for MySQL/MariaDB (#6591)
  fix(editor): Show retry information in execution list only when it exists (#6587)
  fix(Strapi Node): Fix issue with pagination (#4991)
  fix(core): Route `/rest/workflows/new` correctly (no-changelog) (#6572)
  fix(XML Node): Fix issue with not returning valid data (#6565)
  fix(core): Fix credentials test (#6569)
  fix(core): Ensure valid `logger` is passed to every migration (no-changelog) (#6563)
  ...

# Conflicts:
#	packages/design-system/src/components/N8nCallout/Callout.vue
#	packages/editor-ui/src/App.vue
#	packages/editor-ui/src/plugins/i18n/locales/en.json
#	packages/editor-ui/src/stores/ui.store.ts
MiloradFilipovic added a commit that referenced this pull request Jul 7, 2023
* ADO-814-trial-banner: (58 commits)
  ⚡ Updating permanent dismiss logic to work with all banners
  fix(editor): Make Source control branch select required (#6619)
  ⚡ Minor refactoring of banner logic
  refactor(core): Load `cookieParser` middleware only once (no-changelog) (#6614)
  fix(editor): Prevent keyboard shortcuts to edit workflows in readonly mode (#6613)
  fix(editor): Show appropriate empty workflow list content when instance environment is readonly (#6610)
  ✨ Implemented permanent/temporary dismiss logic for v1 flag
  ci: Fix linting issues (no-changelog)
  fix(Google Drive Node): Fix regex in file RLC (#6607)
  feat(HTML Node): 'Convert to table operation (#6540)
  ci: e2e tests don't need to enable saving manual executions
  🚀 Release 1.0.1 (#6606)
  fix(core): Make node execution order configurable, and backward-compatible (#6507)
  🚀 Release 0.236.0 (#6602)
  fix: Fix migrations for postgres and mysql (no-changelog) (#6600)
  fix(core): Improve the performance of last 2 sqlite migrations (#6522)
  refactor: Switch cloud plan retrieval endpoint (no-changelog) (#6529)
  fix(Salesforce Node): Fix typo for adding a contact to a campaign (#6598)
  ci: Hide v1 banner in e2e tests to fix 2-credentials.cy.ts (no-changelog)
  fix(editor): Restore expression completions (#6566)
  ...
@github-actions github-actions bot mentioned this pull request Jul 12, 2023
@github-actions github-actions bot mentioned this pull request Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
n8n team Authored by the n8n team node/improvement New feature or request Released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants