Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(editor): Open template credential setup from collection #7882

Merged
merged 2 commits into from
Nov 30, 2023

Conversation

tomi
Copy link
Collaborator

@tomi tomi commented Nov 30, 2023

Open the template credential setup when using a template from the collection view.

NOTE! This feature is still behind a feature flag. To test, set:

localStorage.setItem('template-credentials-setup', 'true')
Kapture.2023-11-30.at.12.11.01.mp4

@n8n-assistant n8n-assistant bot added n8n team Authored by the n8n team ui Enhancement in /editor-ui or /design-system labels Nov 30, 2023
cypress/e2e/34-template-credentials-setup.cy.ts Outdated Show resolved Hide resolved
cypress/pages/template-collection.ts Outdated Show resolved Hide resolved
params: { id: templateId },
};

if (inNewBrowserTab) {
Copy link
Contributor

Choose a reason for hiding this comment

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

😻

@tomi tomi changed the title feat(editor): open template credential setup from collection feat(editor): Open template credential setup from collection Nov 30, 2023
@tomi tomi requested a review from mutdmour November 30, 2023 11:44
@tomi
Copy link
Collaborator Author

tomi commented Nov 30, 2023

@mutdmour comments addressed, ready for re-review

Copy link

cypress bot commented Nov 30, 2023

1 flaky test on run #3136 ↗︎

0 296 5 0 Flakiness 1

Details:

🌳 🖥️ browsers:node18.12.0-chrome107 🤖 tomi 🗃️ e2e/*
Project: n8n Commit: a1e7f6e38d
Status: Passed Duration: 06:44 💡
Started: Nov 30, 2023 11:53 AM Ended: Nov 30, 2023 12:00 PM
Flakiness  cypress/e2e/12-canvas.cy.ts • 1 flaky test

View Output Video

Test Artifacts
Canvas Node Manipulation and Navigation > should preserve connections after rename & node-view switch Screenshots Video

Review all test suite changes for PR #7882 ↗︎

Copy link
Contributor

✅ All Cypress E2E specs passed

@tomi tomi merged commit 627ddb9 into master Nov 30, 2023
@tomi tomi deleted the fix-open-template-credential-setup-from-collections branch November 30, 2023 12:09
@github-actions github-actions bot mentioned this pull request Dec 6, 2023
ivov added a commit that referenced this pull request Dec 6, 2023
#
[1.20.0](https://github.com/n8n-io/n8n/compare/n8n@1.19.0...n8n@1.20.0)
(2023-12-06)


### Bug Fixes

* **AWS DynamoDB Node:** Improve error message parsing
([#7793](#7793))
([5ba5ed8](5ba5ed8))
* **core:** Allow grace period for binary data deletion after manual
execution ([#7889](#7889))
([61d8aeb](61d8aeb))
* **core:** Consolidate ownership and sharing data on workflows and
credentials ([#7920](#7920))
([38b88b9](38b88b9))
* **core:** Fix hard deletes stopping if database query throws
([#7848](#7848))
([46dd4d3](46dd4d3))
* **core:** Make sure mfa secret and recovery codes are not returned on
login ([#7936](#7936))
([f5502cc](f5502cc))
* **editor:** Fix deletion of last execution at execution preview
([#7883](#7883))
([ce2d388](ce2d388))
* **editor:** Replace isInstanceOwner checks with scopes where
applicable ([#7858](#7858))
([132d691](132d691))
* **Google Sheets Node:** Fix issue with paired items not being set
correctly ([#7862](#7862))
([5207a2f](5207a2f))
* **Notion Node:** Fix broken Notion node parameters
([#7864](#7864))
([51d1f5b](51d1f5b)),
closes [#7791](#7791)


### Features

* **BambooHR Node:** Add support for Only Current on company reports
([#7878](#7878))
([4175801](4175801))
* **core:** Allow admin creation
([#7837](#7837))
([476806e](476806e))
* **editor:** Add sections to create node panel
([#7831](#7831))
([39fa8d2](39fa8d2))
* **editor:** Open template credential setup from collection
([#7882](#7882))
([627ddb9](627ddb9))
* **editor:** Select credentials in template setup if theres only one
([#7879](#7879))
([fe3417a](fe3417a))


### Performance Improvements

* **editor:** Improve node rendering performance when opening large
workflows ([#7904](#7904))
([a8049a0](a8049a0))
* **editor:** Improve performance when opening large workflows with node
issues ([#7901](#7901))
([4bd7ae2](4bd7ae2))

Co-authored-by: ivov <ivov@users.noreply.github.com>
@janober
Copy link
Member

janober commented Dec 6, 2023

Got released with n8n@1.20.0

@github-actions github-actions bot mentioned this pull request Dec 13, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
n8n team Authored by the n8n team Released ui Enhancement in /editor-ui or /design-system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants