fix(core): Fix `frontend.settings` external hook execution #7496

RicardoE105 · 2023-10-23T13:41:48Z

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

github-actions · 2023-10-23T13:42:05Z

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.

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

packages/cli/src/Server.ts

cypress · 2023-10-23T14:00:48Z

Passing run #2551 ↗︎

0	258	0	0	0

Details:

🌳 🖥️ browsers:node18.12.0-chrome107 🤖 RicardoE105 🗃️ e2e/*
Project: n8n	Commit: `d9c55a378d`
Status: Passed	Duration: 09:42 💡
Started: Oct 23, 2023 3:07 PM	Ended: Oct 23, 2023 3:16 PM

Review all test suite changes for PR #7496 ↗︎

github-actions · 2023-10-23T14:00:56Z

✅ All Cypress E2E specs passed

codecov · 2023-10-23T14:08:38Z

Codecov Report

Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (b6de910) 33.55% compared to head (d9c55a3) 33.55%.
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #7496   +/-   ##
=======================================
  Coverage   33.55%   33.55%           
=======================================
  Files        3399     3399           
  Lines      207444   207455   +11     
  Branches    22417    22415    -2     
=======================================
+ Hits        69602    69621   +19     
+ Misses     136714   136706    -8     
  Partials     1128     1128

Files	Coverage Δ
packages/cli/src/services/frontend.service.ts	`0.00% <0.00%> (ø)`

... and 7 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

This reverts commit c370222.

packages/cli/src/services/frontend.service.ts

netroy · 2023-10-23T14:32:07Z

packages/cli/src/services/frontend.service.ts

@@ -198,7 +198,7 @@ export class FrontendService {
 		this.writeStaticJSON('credentials', credentials);
 	}

-	async getSettings(): Promise<IN8nUISettings> {
+	getSettings(): IN8nUISettings {


While we are at it, I think we should add some simple unit tests to cover this case.

Will do after all the metings 💀

netroy

Let's do tests in another PR, so that we can release this without rushing the tests.

github-actions · 2023-10-23T15:17:12Z

✅ All Cypress E2E specs passed

## [1.12.2](https://github.com/n8n-io/n8n/compare/[email protected]@1.12.2) (2023-10-24) ### Bug Fixes * **core:** Fix `frontend.settings` external hook execution ([#7496](#7496)) ([e8294b1](e8294b1)) * **MQTT Trigger Node:** Fix node causing a start up hang when active ([#7498](#7498)) ([ede3047](ede3047)) * **Redis Node:** Fix adding sets data types ([#7444](#7444)) ([27b5543](27b5543)), closes [#6339](#6339) Co-authored-by: netroy <[email protected]>

janober · 2023-10-24T10:36:34Z

Got released with [email protected]

# [1.13.0](https://github.com/n8n-io/n8n/compare/[email protected]@1.13.0) (2023-10-25) ### Bug Fixes * **core:** Do not return `inviteAcceptUrl` in response if email was sent ([#7465](#7465)) ([55c6a1b](55c6a1b)) * **core:** Ensure nodes post-processors run in the correct order ([#7500](#7500)) ([6f45298](6f45298)) * **core:** Fix `frontend.settings` external hook execution ([#7496](#7496)) ([774fe20](774fe20)) * **core:** Handle gzip and deflate compressed request payloads ([#7461](#7461)) ([83762e0](83762e0)) * **core:** Reduce logging overhead for levels that do not output ([#7479](#7479)) ([76c0481](76c0481)) * **Customer.io Node:** Fix api endpoint when using EU region ([#7485](#7485)) ([519680c](519680c)) * **editor:** Allow importing the same workflow multiple times ([#7458](#7458)) ([3c0a166](3c0a166)) * **editor:** Fix canvas selection breaking after interacting with node actions ([#7466](#7466)) ([bc47365](bc47365)) * **editor:** Fix connections disappearing after reactivating canvas and renaming a node ([#7483](#7483)) ([450e0cc](450e0cc)) * **Google Sheets Node:** Append or update runs forever when without column headers ([#7463](#7463)) ([ab6a9bb](ab6a9bb)) * **Microsoft SQL Node:** Prevent SQL injection ([#7467](#7467)) ([a739245](a739245)) * **MQTT Trigger Node:** Fix node causing a start up hang when active ([#7498](#7498)) ([baecb93](baecb93)) * **MySQL Node:** Resolve expressions in v1 ([#7464](#7464)) ([5c46bb0](5c46bb0)) * **Redis Node:** Fix adding sets data types ([#7444](#7444)) ([4e66023](4e66023)) * **Spreadsheet File Node:** Fix include empty cells not working with v2 ([#7505](#7505)) ([05e6f2a](05e6f2a)) ### Features * **core:** Add support for oauth based service accounts with UM SMTP ([#7311](#7311)) ([647372b](647372b)) * **editor:** Add PH tracking to event ([#7511](#7511)) ([c47d27d](c47d27d)) * **Facebook Lead Ads Trigger Node:** Add Facebook Lead Ads Trigger Node ([#7113](#7113)) ([ac814a9](ac814a9)) * **Ghost Node:** Add support for lexical format ([#7488](#7488)) ([7b1973c](7b1973c)) * **RSS Feed Trigger Node:** Add RSS feed trigger node ([#7386](#7386)) ([689360e](689360e)) Co-authored-by: netroy <[email protected]>

* master: (30 commits) 🚀 Release 1.14.0 (#7514) ci: Fix oclif manifest generation feat(Switch Node): Add support for infinite Switch outputs (#7499) 🚀 Release 1.13.0 (#7512) fix(core): Ensure nodes post-processors run in the correct order (#7500) feat(editor): Add PH tracking to event (#7511) fix(core): Fix workflow activation with history and workflow history for EE (no-changelog) (#7508) refactor(core): Make executions pruning more resilient (#7480) fix(Spreadsheet File Node): Fix include empty cells not working with v2 (#7505) fix(core): Create instance settings directory recursively (no-changelog) (#7506) fix(Microsoft SQL Node): Prevent SQL injection (#7467) refactor(core): Make pruning via lifecycle configuration in S3 mode mandatory (#7482) fix(core): Always derive `instanceId` from the encryption key (no-changlog) (#7501) fix(MQTT Trigger Node): Fix node causing a start up hang when active (#7498) feat: Collect usage metrics on license renewal (no-changelog) (#7486) fix(core): Fix `frontend.settings` external hook execution (#7496) fix(Redis Node): Fix adding sets data types (#7444) fix: Save new version of the workflow instead of the previous (no-changelog) (#7428) refactor(core): Abstract away InstanceSettings and `encryptionKey` into injectable services (no-changelog) (#7471) fix(Customer.io Node): Fix api endpoint when using EU region (#7485) ...

await promise before passing settings

c449a62

RicardoE105 changed the title ~~await promise before passing settings~~ fix: await promise before passing settings so they n8nMetadata can be injected Oct 23, 2023

RicardoE105 changed the title ~~fix: await promise before passing settings so they n8nMetadata can be injected~~ fix: await promise before passing settings so they n8nMetadata can be injected (no-changelog) Oct 23, 2023

RicardoE105 requested a review from mutdmour October 23, 2023 13:42

RicardoE105 changed the title ~~fix: await promise before passing settings so they n8nMetadata can be injected (no-changelog)~~ fix: Await promise before passing settings so they n8nMetadata can be injected (no-changelog) Oct 23, 2023

mutdmour previously approved these changes Oct 23, 2023

View reviewed changes

netroy reviewed Oct 23, 2023

View reviewed changes

packages/cli/src/Server.ts Outdated Show resolved Hide resolved

n8n-assistant bot added core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team labels Oct 23, 2023

make getSettings non async

c370222

RicardoE105 dismissed mutdmour’s stale review via c370222 October 23, 2023 14:08

RicardoE105 requested a review from netroy October 23, 2023 14:08

RicardoE105 added 2 commits October 23, 2023 10:14

Revert "make getSettings non async"

37effdc

This reverts commit c370222.

make getSettings non async

5fb4064

netroy reviewed Oct 23, 2023

View reviewed changes

packages/cli/src/services/frontend.service.ts Show resolved Hide resolved

netroy reviewed Oct 23, 2023

View reviewed changes

packages/cli/src/services/frontend.service.ts Outdated Show resolved Hide resolved

RicardoE105 requested a review from netroy October 23, 2023 14:19

netroy reviewed Oct 23, 2023

View reviewed changes

apply adi's changes

d9c55a3

netroy changed the title ~~fix: Await promise before passing settings so they n8nMetadata can be injected (no-changelog)~~ fix(core): Fix frontend.settings external hook execution Oct 23, 2023

netroy approved these changes Oct 23, 2023

View reviewed changes

RicardoE105 merged commit 774fe20 into master Oct 23, 2023
23 checks passed

RicardoE105 deleted the ado-1245-bug-1121-does-not-set-n8nmetadata-in-settings branch October 23, 2023 15:46

netroy pushed a commit that referenced this pull request Oct 24, 2023

fix(core): Fix frontend.settings external hook execution (#7496)

e8294b1

github-actions bot mentioned this pull request Oct 24, 2023

🚀 Release 1.12.2 #7504

Merged

janober added the Released label Oct 24, 2023

github-actions bot mentioned this pull request Oct 25, 2023

🚀 Release 1.13.0 #7512

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(core): Fix `frontend.settings` external hook execution #7496

fix(core): Fix `frontend.settings` external hook execution #7496

RicardoE105 commented Oct 23, 2023

github-actions bot commented Oct 23, 2023

cypress bot commented Oct 23, 2023 •

edited

Loading

github-actions bot commented Oct 23, 2023

codecov bot commented Oct 23, 2023 •

edited

Loading

netroy Oct 23, 2023

RicardoE105 Oct 23, 2023 •

edited

Loading

netroy left a comment

github-actions bot commented Oct 23, 2023

janober commented Oct 24, 2023

fix(core): Fix frontend.settings external hook execution #7496

fix(core): Fix frontend.settings external hook execution #7496

Conversation

RicardoE105 commented Oct 23, 2023

github-actions bot commented Oct 23, 2023

cypress bot commented Oct 23, 2023 • edited Loading

Passing run #2551 ↗︎

Review all test suite changes for PR #7496 ↗︎

github-actions bot commented Oct 23, 2023

codecov bot commented Oct 23, 2023 • edited Loading

Codecov Report

netroy Oct 23, 2023

Choose a reason for hiding this comment

RicardoE105 Oct 23, 2023 • edited Loading

Choose a reason for hiding this comment

netroy left a comment

Choose a reason for hiding this comment

github-actions bot commented Oct 23, 2023

janober commented Oct 24, 2023

fix(core): Fix `frontend.settings` external hook execution #7496

fix(core): Fix `frontend.settings` external hook execution #7496

cypress bot commented Oct 23, 2023 •

edited

Loading

codecov bot commented Oct 23, 2023 •

edited

Loading

RicardoE105 Oct 23, 2023 •

edited

Loading