Skip to content

Add CC property to email - #35

Merged
janober merged 1 commit into
n8n-io:masterfrom
Libermentix:master
Oct 9, 2019
Merged

janober merged 1 commit into
n8n-io:masterfrom
Libermentix:master

Conversation

@Libermentix

Copy link
Copy Markdown
Contributor

Thanks for such a great tool! We make use of it on a regular basis.

I took the liberty to extend the e-mail CC property to the Node's settings.

As this is my first PR on your repo, let me know if this suffices the way it is.

@janober
janober merged commit d5bf00c into n8n-io:master Oct 9, 2019
@janober

janober commented Oct 9, 2019

Copy link
Copy Markdown
Member

Happy to welcome you as n8n contributor! Thanks a lot for the addition and great to know that n8n is helpful for you!

Sorry for the late merge. Everything was a little bit crazy with the release. It will be faster the next time.

@janober

janober commented Oct 9, 2019

Copy link
Copy Markdown
Member

Will release a version with the addition within the next hours.

@Libermentix

Copy link
Copy Markdown
Contributor Author

cool! happy to be on-board.

@janober

janober commented Oct 9, 2019

Copy link
Copy Markdown
Member

That makes you actually contributor Nr. 5 ;-)

Btw. did just release nan@0.24.0

It is on npm but docker-image is still building.

netroy pushed a commit that referenced this pull request Mar 20, 2025
DG-Lordwilly55 pushed a commit to DG-Lordwilly55/n8nVanguardia that referenced this pull request May 29, 2025
…ry-or-body

Revert "GroupJid por Query ou por Body"
@Joffcom
Joffcom requested a review from dlavrenuek March 3, 2026 15:15
n8n-assistant Bot pushed a commit that referenced this pull request Mar 25, 2026
* fix: Improve property handling in ldap node search operation (#33)

* fix: Escape special characters in ldap search

* add tests for ldap node

* Apply suggestion

* add tests

* fix: Improve query operations in sql agend node (#34)

* fix: Improve query operations in sql agend node

* improve keyword detection

* refactor(Data Table Node): Return early if Order By column does not exist (#37)

* fix: Improve domain restrictions for HTTP Request Node (#35)

* fix: Block domains that are not whitelisted on redirects and pagination requests

* test: request-helper-functions tests

* test: GenericFunctions tests for HttpRequest node

* refactor: Tests

* fix: Make prop required

* fix: Make sure only one credential can be set in HTTP request node v1

* refactor: Clean up Chat Trigger template rendering (#36)

* fix(Merge Node): Improve sandbox robustness (#38)

* fix(Merge Node): Improve sandbox robustness

Made-with: Cursor

* Fix formatting

* fix(core): Add missing headers to binary data responses (#41)

Set Content-Disposition and Content-Security-Policy headers on all
binary data endpoint responses, consistent with how other endpoints
already handle these headers.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

* refactor(core): Improve credential resolution logic (no-changelog) (#43)

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

* refactor(core): Improve http request debugging (#45)

* feat(core): Improve variables endpoint (#47)

* fix(Google Workspace Admin Node): Validate additional fields for user creation and update (#46)

* fix: Validate properties before setting them

* fix: Handle when schema name was invalid

* test: Add tests

* fix: Use Object.hasOwn

* fix: Improve mcp oAuth client registration (#48)

* fix: Improve mcp oAuth client registration
* 👌 Prevent race condition, remove hard-coded name constraint
* 👌 Remove `MAX_GRANT_TYPES`, move `MAX_REGISTERED_CLIENTS` to env
* Update packages/cli/src/modules/mcp/mcp-oauth-service.ts

Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
* 👕 Add missing mocks
* 👕 Add missing constants
* ✔️ Update tests
* refactor: Extract client limit enforcement into dedicated method
* refactor: Resolve McpSettingsService at module level in OAuth controller
* revert: Remove column length constraint that requires a migration
---------
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
Co-authored-by: Ricardo Espinoza <ricardo@n8n.io>

* run format

* Remove circural dependencies caused by merges

* Fix axios-utils tests

* Only run benchmark in public repository

* Fix unit tests after merge

---------

Co-authored-by: Dimitri Lavrenük <20122620+dlavrenuek@users.noreply.github.com>
Co-authored-by: Charlie Kolb <charlie@n8n.io>
Co-authored-by: RomanDavydchuk <roman.davydchuk@n8n.io>
Co-authored-by: Jaakko Husso <jaakko@n8n.io>
Co-authored-by: Dawid Myslak <dawid.myslak@gmail.com>
Co-authored-by: Svetoslav Dekov <svetoslav.dekov@n8n.io>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Guillaume Jacquart <jacquart.guillaume@gmail.com>
Co-authored-by: Sandra Zollner <sandra.zollner@n8n.io>
Co-authored-by: Milorad FIlipović <milorad@n8n.io>
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
Co-authored-by: Ricardo Espinoza <ricardo@n8n.io>
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.

2 participants