Milorad FIlipović
327cc8df73
fix(editor): Send template id as a number in telemetry events ( #8484 )
2024-01-30 10:37:06 +01:00
Csaba Tuncsik
ad4b298be3
fix(editor): Fix workflows filter resetting ( #8411 )
2024-01-26 06:55:08 +01:00
कारतोफ्फेलस्क्रिप्ट™
d6deceacde
refactor(core): Remove roleId indirection (no-changelog) ( #8413 )
2024-01-24 13:38:57 +01:00
Alex Grozav
b6d775768f
feat(editor): Migrate moveNodeWorkflow mixin to useCanvasPanning composable ( #8322 )
2024-01-19 16:27:44 +02:00
Csaba Tuncsik
83228e26fb
fix(editor): Add pinned data for freshly added nodes ( #8323 )
2024-01-19 14:44:54 +01:00
Tomi Turtiainen
e606e841ee
fix(editor): Use web native <a> element in nav menus ( #8385 )
2024-01-19 12:52:39 +02:00
Tomi Turtiainen
1aa35b190a
fix: Set '@typescript-eslint/return-await' rule to 'always' for FE (no-changelog) ( #8373 )
2024-01-18 11:28:01 +02:00
Ricardo Espinoza
4277e92ec0
feat(editor): Add new /templates/search endpoint ( #8227 )
...
Updating n8n front-end to use the new search endpoint powered by TypeSense.
Endpoint is deployed on staging API so, in order to test it, use this env var:
```export N8N_TEMPLATES_HOST=https://api-staging.n8n.io/api ```
**NOTE**: This PR should not be merged until [backend changes](https://github.com/n8n-io/creators-site/pull/118 ) are merged and released.
## Related tickets and issues
https://linear.app/n8n/issue/ADO-1555/update-in-app-search-to-work-with-the-new-back-end
## Review / Merge checklist
- [x] PR title and summary are descriptive. **Remember, the title automatically goes into the changelog. Use `(no-changelog)` otherwise.** ([conventions](https://github.com/n8n-io/n8n/blob/master/.github/pull_request_title_conventions.md ))
- [ ] Tests included.
> A bug is not considered fixed, unless a test is added to prevent it from happening again.
> A feature is not complete without tests.
2024-01-15 16:19:24 -05:00
Ricardo Espinoza
6b92d49ea5
fix(editor): Fix issue with synchronization table on LDAP not loading data ( #8327 )
...
## Summary
When we migrated to Vue3, we updated to a new component and did not provide the right props.
before:
https://www.loom.com/share/ce12b21fdda44176b3e949a00c582252
now:
https://www.loom.com/share/354d1dcb51bd47a9910d5bfe34c7c9b8
## Related tickets and issues
https://linear.app/n8n/issue/PAY-1134/ldap-synchronization-table-in-ui-remains-empty-even-after-successful
## Review / Merge checklist
- [x] PR title and summary are descriptive. **Remember, the title automatically goes into the changelog. Use `(no-changelog)` otherwise.** ([conventions](https://github.com/n8n-io/n8n/blob/master/.github/pull_request_title_conventions.md ))
2024-01-15 09:03:10 -05:00
Tomi Turtiainen
135553bd6b
feat: Redirect users without feature flag from template cred setup (no-changelog) ( #8302 )
2024-01-12 12:10:39 +02:00
Alex Grozav
1824ff6a13
fix(editor): Add back missing computed prop (no-changelog) ( #8306 )
2024-01-11 16:56:42 +02:00
Csaba Tuncsik
3b8ccb9fb9
fix(editor): Properly set colors for connections and labels on nodes with pinned data ( #8209 )
...
Co-authored-by: Alex Grozav <alex@grozav.com >
2024-01-11 14:03:23 +01:00
Tomi Turtiainen
d877d3ce92
fix: Adjust cred setup telemetry (no-changelog) ( #8294 )
2024-01-10 19:41:25 +02:00
Alex Grozav
d32e6a60da
feat(editor): Migrate deviceSupportHelpers mixin to useDeviceSupport composable (no-changelog) ( #8289 )
2024-01-10 16:42:01 +02:00
Iván Ovejero
0dabe5c74e
fix: Fix user reinvites on FE and BE ( #8261 )
2024-01-09 13:52:34 +01:00
Alex Grozav
af49e95cc7
feat: Add Chat Trigger node ( #7409 )
...
Signed-off-by: Oleg Ivaniv <me@olegivaniv.com >
Co-authored-by: Jan Oberhauser <jan.oberhauser@gmail.com >
Co-authored-by: Jesper Bylund <mail@jesperbylund.com >
Co-authored-by: OlegIvaniv <me@olegivaniv.com >
Co-authored-by: Deborah <deborah@starfallprojects.co.uk >
Co-authored-by: Jan Oberhauser <janober@users.noreply.github.com >
Co-authored-by: Jon <jonathan.bennetts@gmail.com >
Co-authored-by: कारतोफ्फेलस्क्रिप्ट™ <aditya@netroy.in >
Co-authored-by: Michael Kret <88898367+michael-radency@users.noreply.github.com >
Co-authored-by: Giulio Andreini <andreini@netseven.it >
Co-authored-by: Mason Geloso <Mason.geloso@gmail.com >
Co-authored-by: Mason Geloso <hone@Masons-Mac-mini.local >
Co-authored-by: Mutasem Aldmour <mutasem@n8n.io >
2024-01-09 13:11:39 +02:00
Tomi Turtiainen
b6c42cc084
fix(editor): Unify canvas button positioning ( #8258 )
2024-01-08 14:44:56 +02:00
Alex Grozav
8c8caac4e8
feat(editor): Migrate debounce mixin to useDebounce composable (no-changelog) ( #8244 )
2024-01-08 14:00:49 +02:00
Tomi Turtiainen
7899f7aca5
fix: Adjust telemetry from canvas (no-changelog) ( #8252 )
2024-01-08 13:56:16 +02:00
Tomi Turtiainen
3cf6704dbb
feat: Enable cred setup for workflows created from templates (no-changelog) ( #8240 )
...
## Summary
Enable users to open credential setup for workflows that have been
created from templates if they skip it.
Next steps (will be their own PRs):
- Add telemetry events
- Add e2e test
- Hide the button when user sets up all the credentials
- Change the feature flag to a new one
## Related tickets and issues
https://linear.app/n8n/issue/ADO-1637/feature-support-template-credential-setup-for-http-request-nodes-that
2024-01-05 18:07:57 +02:00
Ricardo Espinoza
df5d07bcb8
feat(editor): Update copy: Execute --> Test ( #8137 )
...
## Summary
Title self explanatory
## Related tickets and issues
https://linear.app/n8n/issue/ADO-129/update-copy-execute-test
## Review / Merge checklist
- [x] PR title and summary are descriptive. **Remember, the title
automatically goes into the changelog. Use `(no-changelog)` otherwise.**
([conventions](https://github.com/n8n-io/n8n/blob/master/.github/pull_request_title_conventions.md ))
2024-01-05 10:23:51 -05:00
oleg
184ed8e17d
refactor: Migrate genericHelpers mixin to composable ( #8220 )
...
## Summary
- Moved out canvas loading handling to canvas store
- Tag editable routes via meta to remove router dependency from generic
helpers
- Replace all occurrences of `genericHelpers` mixin with composable and
audit usage
- Moved out `isRedirectSafe` and `getRedirectQueryParameter` out of
genericHelpers to remove dependency on router
Removing the router dependency is important, because `useRouter` and
`useRoute` compostables are only available if called from component
instance. So if composable is nested within another composable, we
wouldn't be able to use these. In this case we'd always need to inject
the router and pass it through several composables. That's why I moved
the `readonly` logic to router meta and `isRedirectSafe` and
`getRedirectQueryParameter` out as they were only used in a single
component.
---------
Signed-off-by: Oleg Ivaniv <me@olegivaniv.com >
2024-01-05 12:23:28 +01:00
Tomi Turtiainen
6e78d2346e
fix: Adjust 'use template' feature telemetry (no-changelog) ( #8232 )
...
- Add extra params to 'User closed cred setup' event
- Fire 'User closed cred setup' only when template has creds
- Skip cred setup page if template has no creds required
- Fire 'User inserted workflow template' also in cred setup
2024-01-05 11:52:10 +02:00
Alex Grozav
b50d8058cf
feat(editor): Migrate pinData mixin to usePinnedData composable (no-changelog) ( #8207 )
...
## Summary
Required as part of NodeView refactoring:
- Migrates `pinData` mixin to `usePinnedData` composable.
- Adds `useActiveNode` and `useNodeType` composables
## Related tickets and issues
https://linear.app/n8n/issue/N8N-6355/pindata
## Review / Merge checklist
- [x] PR title and summary are descriptive. **Remember, the title
automatically goes into the changelog. Use `(no-changelog)` otherwise.**
([conventions](https://github.com/n8n-io/n8n/blob/master/.github/pull_request_title_conventions.md ))
- [x] [Docs updated](https://github.com/n8n-io/n8n-docs ) or follow-up
ticket created.
- [x] Tests included.
> A bug is not considered fixed, unless a test is added to prevent it
from happening again.
> A feature is not complete without tests.
2024-01-04 11:22:56 +02:00
Tomi Turtiainen
cd3f5b5b1f
fix: Fix template credential setup for nodes that dont have credentials ( #8208 )
...
Fix template credential setup for templates whose workflow includes
nodes that require credentials but the workflow definition does not have
them defined. Like for example
https://n8n.io/workflows/1344-save-email-attachments-to-nextcloud/
2024-01-04 10:21:36 +02:00
कारतोफ्फेलस्क्रिप्ट™
cfe9525dd4
fix(core): Better input validation for the changeRole endpoint ( #8189 )
...
also refactored the code to
1. stop passing around `scope === 'global'`, since this code can be used
only for changing globalRole.
2. leak less details when input validation fails.
## Review / Merge checklist
- [x] PR title and summary are descriptive
- [x] Tests included
2024-01-03 09:33:35 +01:00
Alex Grozav
f5a4bfe40c
feat(editor): Migrate copyPaste mixin to composables (no-changelog) ( #8179 )
2023-12-29 12:13:24 +02:00
कारतोफ्फेलस्क्रिप्ट™
68cff4c59e
refactor(editor): Improve linting for component and prop names (no-changelog) ( #8169 )
2023-12-28 09:49:58 +01:00
Milorad FIlipović
517b050d0a
feat(editor): Add template Id to workflow metadata ( #8088 )
...
## Summary
Adding a link between the workflow and the template it originated from
by saving `templateId` in the workflow metadata
## Related tickets and issues
ADO-1537
## Review / Merge checklist
- [x] PR title and summary are descriptive. **Remember, the title
automatically goes into the changelog. Use `(no-changelog)` otherwise.**
([conventions](https://github.com/n8n-io/n8n/blob/master/.github/pull_request_title_conventions.md ))
- [ ] [Docs updated](https://github.com/n8n-io/n8n-docs ) or follow-up
ticket created.
- [x] Tests included.
> A bug is not considered fixed, unless a test is added to prevent it
from happening again.
> A feature is not complete without tests.
2023-12-22 15:07:05 +01:00
Ricardo Espinoza
e9c7fd7397
fix: Show public API upgrade CTA when feature is not enabled ( #8109 )
...
## Summary
> Describe what the PR does and how to test. Photos and videos are
recommended.
Shows the public API upgrade CTA when the feature is not enabled. Now
trialing users in cloud would see the API on the settings menu and can
upgrade from there.
When public API feature disabled:
<img width="1863" alt="image"
src="https://github.com/n8n-io/n8n/assets/16496553/a9052f6b-151f-4ebb-80df-5ff2fa643e85 ">
When public API feature enabled with no API key:
<img width="1861" alt="image"
src="https://github.com/n8n-io/n8n/assets/16496553/fdfe7296-425e-4410-a8cb-f25535bc9df4 ">
When public API feature enabled with API key:
<img width="1423" alt="image"
src="https://github.com/n8n-io/n8n/assets/16496553/a8426cbf-ff73-43c4-8c60-b00274bca46e ">
## Related tickets and issues
[> Include links to **Linear ticket** or Github issue or Community forum
post. Important in order to close *automatically* and provide context to
r](https://linear.app/n8n/issue/ADO-1282/feature-api-page-missing-for-trial-users )eviewers.
## Review / Merge checklist
- [x] PR title and summary are descriptive. **Remember, the title
automatically goes into the changelog. Use `(no-changelog)` otherwise.**
([conventions](https://github.com/n8n-io/n8n/blob/master/.github/pull_request_title_conventions.md ))
- [x] Tests included.
> A bug is not considered fixed, unless a test is added to prevent it
from happening again.
> A feature is not complete without tests.
2023-12-21 09:06:54 -05:00
Iván Ovejero
d917dfe9f8
refactor(editor): Remove tracking for SAML test success (no-changelog) ( #8108 )
...
Follow-up to: https://github.com/n8n-io/n8n/pull/8102
It turns out that SAML test success cannot be tracked on the FE. We
might track on the BE, but after further discussion with Product, we do
not really need to track this property, so this PR simply removes it.
2023-12-20 15:23:00 +01:00
Elias Meire
8df49e134d
fix(editor): Make keyboard shortcuts more strict; don't accept extra Ctrl/Alt/Shift keys ( #8024 )
...
## Summary
Keyboard shortcuts such as Shift+S get triggered even when other keys
(Ctrl/Alt) are pressed, creating conflicts with browser or OS shortcuts.
This PR makes keyboard shortcuts more strict: the exact combination
needs to be pressed, no extra keys allowed.
--> Ctrl+Shift+S will no longer trigger the Shift+S shortcut to add a
sticky note
## Related tickets and issues
https://n8nio.slack.com/archives/C035KBDA917/p1702545933647959
2023-12-20 12:06:49 +01:00
Iván Ovejero
faadfd6d4a
refactor(editor): Add telemetry for SSO/SAML (no-changelog) ( #8102 )
...
https://linear.app/n8n/issue/PAY-1142
2023-12-20 11:41:39 +01:00
कारतोफ्फेलस्क्रिप्ट™
9d22c7a278
feat(editor): Gracefully ignore invalid payloads in postMessage handler ( #8096 )
...
It's possible for browser extensions to send `message` events to an n8n window.
This change improves the already existing check to ignore invalid messages.
Fixes https://community.n8n.io/t/errors-in-the-browser-console-on-n8n-1-version/33910
2023-12-19 15:20:29 +01:00
Milorad FIlipović
36a923cf7b
feat(editor): Add lead enrichment suggestions to workflow list ( #8042 )
...
## Summary
We want to show lead enrichment template suggestions to cloud users that
agreed to this. This PR introduces the front-end part of this feature
- Handoff document
- Figma Hi-fi
- [How to
test](https://linear.app/n8n/issue/ADO-1549/[n8n-fe]-update-workflows-list-page-to-show-fake-door-templates#comment-b6644c99 )
Tests are being worked on in a separate PR
## Related tickets and issues
Fixes ADO-1546
Fixes ADO-1549
Fixes ADO-1604
## Review / Merge checklist
- [ ] PR title and summary are descriptive. **Remember, the title
automatically goes into the changelog. Use `(no-changelog)` otherwise.**
([conventions](https://github.com/n8n-io/n8n/blob/master/.github/pull_request_title_conventions.md ))
- [ ] [Docs updated](https://github.com/n8n-io/n8n-docs ) or follow-up
ticket created.
- [ ] Tests included.
> A bug is not considered fixed, unless a test is added to prevent it
from happening again.
> A feature is not complete without tests.
---------
Co-authored-by: ricardo <ricardoespinoza105@gmail.com >
2023-12-19 09:10:03 -05:00
Mutasem Aldmour
63a6e7e034
fix(editor): Update image sizes in template description not to be full width always ( #8037 )
...
## Summary
Update image sizes in template description not to be full width always.
## Related tickets and issues
https://linear.app/n8n/issue/ADO-1506/bug-fix-images-in-template-descriptions
## Review / Merge checklist
- [X] PR title and summary are descriptive. **Remember, the title
automatically goes into the changelog. Use `(no-changelog)` otherwise.**
([conventions](https://github.com/n8n-io/n8n/blob/master/.github/pull_request_title_conventions.md ))
- [ ] [Docs updated](https://github.com/n8n-io/n8n-docs ) or follow-up
ticket created.
- [x] Tests included.
> A bug is not considered fixed, unless a test is added to prevent it
from happening again.
> A feature is not complete without tests.
2023-12-18 17:40:00 +01:00
Iván Ovejero
62ce962ec9
refactor(editor): Add telemetry for workflow history (no-changelog) ( #8072 )
...
https://linear.app/n8n/issue/PAY-1142
2023-12-18 16:10:45 +01:00
Iván Ovejero
8fc9888541
refactor(editor): Add telemetry for workers view ( #8055 )
...
https://linear.app/n8n/issue/PAY-1142
2023-12-18 12:21:13 +01:00
oleg
b780436a6b
perf(editor): Improve canvas rendering performance ( #8022 )
...
## Summary
- Refactor usage of `setSuspendDrawing`, removing it from loops and only
calling it after batch operations are done
- Batch adding of nodes to improve copy/paste and workflow load
performance
- Cache i18n calls
- Debounce connections dragging handler if there are more than 20 nodes
## Related tickets and issues
> Include links to **Linear ticket** or Github issue or Community forum
post. Important in order to close *automatically* and provide context to
reviewers.
- https://community.n8n.io/t/slow-ui-in-big-scenarios/33830/8
---------
Signed-off-by: Oleg Ivaniv <me@olegivaniv.com >
2023-12-14 12:01:00 +01:00
Michael Kret
953a58f18b
feat(n8n Form Trigger Node): Improvements ( #7571 )
...
Github issue / Community forum post (link here to close automatically):
---------
Co-authored-by: कारतोफ्फेलस्क्रिप्ट™ <aditya@netroy.in >
Co-authored-by: Giulio Andreini <andreini@netseven.it >
2023-12-13 17:00:51 +02:00
Tomi Turtiainen
08ee307209
feat(editor): Show template credential setup based on feature flag ( #7989 )
...
Replace the local storage based feature flag with posthog feature flag.
Also:
- Fix bunch of eslint warnings in posthog store
2023-12-11 20:21:10 +02:00
Milorad FIlipović
240d259260
refactor(editor): Rename CollectionCard and CollectionCardCarousel ( #7994 )
...
## Summary
As part of the plan to reuse current `CollectionCarousel` component,
this PR makes it, and the cards it uses, more generic by renaming them
and adds a new property to card component which can be used to hide the
item count.
#### How to test the change:
Until the component that will use this is implmeneted, this can be
tested manually:
1. Run n8n
2. Set `showItemCount` property for workflow collection cards to `false`
3. Check if workflow count is showing (it shouldn't) and if everything
rendering correctly
## Issues fixed
Fixes ADO-1567
## Review / Merge checklist
- [ ] PR title and summary are descriptive. **Remember, the title
automatically goes into the changelog. Use `(no-changelog)` otherwise.**
([conventions](https://github.com/n8n-io/n8n/blob/master/.github/pull_request_title_conventions.md ))
- [ ] [Docs updated](https://github.com/n8n-io/n8n-docs ) or follow-up
ticket created.
- [ ] Tests included.
> A bug is not considered fixed, unless a test is added to prevent it
from happening again. A feature is not complete without tests.
>
> *(internal)* You can use Slack commands to trigger [e2e
tests](https://www.notion.so/n8n/How-to-use-Test-Instances-d65f49dfc51f441ea44367fb6f67eb0a?pvs=4#a39f9e5ba64a48b58a71d81c837e8227 )
or [deploy test
instance](https://www.notion.so/n8n/How-to-use-Test-Instances-d65f49dfc51f441ea44367fb6f67eb0a?pvs=4#f6a177d32bde4b57ae2da0b8e454bfce )
or [deploy early access version on
Cloud](https://www.notion.so/n8n/Cloudbot-3dbe779836004972b7057bc989526998?pvs=4#fef2d36ab02247e1a0f65a74f6fb534e ).
2023-12-11 17:37:26 +01:00
Milorad FIlipović
393afef174
feat(editor): Add option to disable NDV in workflow previews ( #7990 )
...
## Summary
This PR adds a new property to the `WorkflowPreview` component that
controls if the Node Details View can be opened in each workflow preview
or not.
This property is passed on to underlying `iframe` via the `postMessage`
and is used to disable pointer events for nodes and stickies in that
preview.
#### How to test the change:
Until the component that uses this flag is implemented this can be
tested manually. In this example, we'll use `WorkflowPreview` as part of
the `TemplatesWorkflowView` component
1. Pass `:canOpenNDV="false"` to this component
2. Run n8n
3. Open any template preview and check if nodes and stickies are
interactive
4. Open any workflow execution preview -> Nodes and Stickies should be
interactive
## Issues fixed
Fixes ADO-1548
## Review / Merge checklist
- [ ] PR title and summary are descriptive. **Remember, the title
automatically goes into the changelog. Use `(no-changelog)` otherwise.**
([conventions](https://github.com/n8n-io/n8n/blob/master/.github/pull_request_title_conventions.md ))
- [ ] [Docs updated](https://github.com/n8n-io/n8n-docs ) or follow-up
ticket created.
- [ ] Tests included.
> A bug is not considered fixed, unless a test is added to prevent it
from happening again. A feature is not complete without tests.
>
> *(internal)* You can use Slack commands to trigger [e2e
tests](https://www.notion.so/n8n/How-to-use-Test-Instances-d65f49dfc51f441ea44367fb6f67eb0a?pvs=4#a39f9e5ba64a48b58a71d81c837e8227 )
or [deploy test
instance](https://www.notion.so/n8n/How-to-use-Test-Instances-d65f49dfc51f441ea44367fb6f67eb0a?pvs=4#f6a177d32bde4b57ae2da0b8e454bfce )
or [deploy early access version on
Cloud](https://www.notion.so/n8n/Cloudbot-3dbe779836004972b7057bc989526998?pvs=4#fef2d36ab02247e1a0f65a74f6fb534e ).
2023-12-11 15:30:01 +01:00
oleg
35fbc37c8e
refactor(editor): Refactor nodeHelpers mixin to composable ( #7810 )
...
- Convert `nodeHelpers` mixin into composable and fix types
- Replace usage of the mixin with the new composable
- Add missing store imports in components that were dependent on opaque
imports from nodeHelpers mixin
- Refactor the `CollectionParameter` component to the modern script
setup syntax
Github issue / Community forum post (link here to close automatically):
---------
Signed-off-by: Oleg Ivaniv <me@olegivaniv.com >
2023-12-08 16:59:03 +01:00
Csaba Tuncsik
5b2defc867
fix(editor): Return early in ws message handler if no 'command' keyword is found ( #7946 )
...
## Summary
Avoid processing websocket messages not n8n sent (filter on 'command'
keyword)
2023-12-08 13:42:50 +01:00
oleg
dcf12867b3
feat: AI nodes usability fixes + Summarization Chain V2 ( #7949 )
...
Fixes:
- Refactor connection snapping when dragging and enable it also for
non-main connection types
- Fix propagation of errors from sub-nodes
- Fix chat scrolling when sending/receiving messages
- Prevent empty chat messages
- Fix sub-node selected styles
- Fix output names text overflow
Usability improvements:
- Auto-add manual chat trigger for agents & chain nodes
- Various labels and description updates
- Make the output parser input optional for Basic LLM Chain
- Summarization Chain V2 with a simplified document loader & text
chunking mode
#### How to test the change:
Example workflow showcasing different operation mode of the new
summarization chain:
[Summarization_V2.json](https://github.com/n8n-io/n8n/files/13599901/Summarization_V2.json )
## Issues fixed
Include links to Github issue or Community forum post or **Linear
ticket**:
> Important in order to close automatically and provide context to
reviewers
-
https://www.notion.so/n8n/David-Langchain-Posthog-notes-7a9294938420403095f4508f1a21d31d
- https://linear.app/n8n/issue/N8N-7070/ux-fixes-batch
- https://linear.app/n8n/issue/N8N-7071/ai-sub-node-bugs
## Review / Merge checklist
- [x] PR title and summary are descriptive. **Remember, the title
automatically goes into the changelog. Use `(no-changelog)` otherwise.**
([conventions](https://github.com/n8n-io/n8n/blob/master/.github/pull_request_title_conventions.md ))
- [x] [Docs updated](https://github.com/n8n-io/n8n-docs ) or follow-up
ticket created.
- [ ] Tests included.
> A bug is not considered fixed, unless a test is added to prevent it
from happening again. A feature is not complete without tests.
>
> *(internal)* You can use Slack commands to trigger [e2e
tests](https://www.notion.so/n8n/How-to-use-Test-Instances-d65f49dfc51f441ea44367fb6f67eb0a?pvs=4#a39f9e5ba64a48b58a71d81c837e8227 )
or [deploy test
instance](https://www.notion.so/n8n/How-to-use-Test-Instances-d65f49dfc51f441ea44367fb6f67eb0a?pvs=4#f6a177d32bde4b57ae2da0b8e454bfce )
or [deploy early access version on
Cloud](https://www.notion.so/n8n/Cloudbot-3dbe779836004972b7057bc989526998?pvs=4#fef2d36ab02247e1a0f65a74f6fb534e ).
---------
Signed-off-by: Oleg Ivaniv <me@olegivaniv.com >
Co-authored-by: Elias Meire <elias@meire.dev >
2023-12-08 13:42:32 +01:00
Csaba Tuncsik
dbd62a4992
feat: Introduce advanced permissions ( #7844 )
...
This PR introduces the possibility of inviting new users with an `admin`
role and changing the role of already invited users.
Also using scoped permission checks where applicable instead of using
user role checks.
---------
Co-authored-by: Val <68596159+valya@users.noreply.github.com >
Co-authored-by: Alex Grozav <alex@grozav.com >
Co-authored-by: Iván Ovejero <ivov.src@gmail.com >
2023-12-08 12:52:25 +01:00
Tomi Turtiainen
a3ca7c7acd
fix: Disable continue button unless at least 1 cred has been filled (no-changelog) ( #7959 )
...
Disable the continue button in template credential setup unless at least
one credential has been (pre-)filled
2023-12-07 19:35:34 +02:00
Tomi Turtiainen
dc1f14b0be
fix(editor): Send correct telemetry events in credential setup (no-changelog) ( #7926 )
...
Send the `User opened cred setup` and `User closed cred setup` events
specified
[here](https://www.notion.so/n8n/Handoff-a1150c38f6e042db91fdf61c56900967?pvs=4 )
2023-12-07 17:25:33 +02:00
Mutasem Aldmour
04cabafef7
feat(editor): Add AppCues tracking for onboarding event ( #7945 )
...
## Summary
We want to trigger a flow, every time the template onboarding event is
triggered. AppCues no longer supports flows triggering through Segment,
except directly.
#### How to test the change:
1. Open on Cloud, where AppCues scripts are available
2. import workflow through /workflows/onboarding/1946
3. Ensure event is sent.. if flow is enabled, flow should show
<img width="1702" alt="Screenshot 2023-12-07 at 14 39 51"
src="https://github.com/n8n-io/n8n/assets/4711238/e58daae8-4364-41a4-bfda-25edeea6cc69 ">
## Issues fixed
Include links to Github issue or Community forum post or **Linear
ticket**:
> Important in order to close automatically and provide context to
reviewers
https://linear.app/n8n/issue/ADO-1455/bug-appcues-trigger
## Review / Merge checklist
- [X] PR title and summary are descriptive. **Remember, the title
automatically goes into the changelog. Use `(no-changelog)` otherwise.**
([conventions](https://github.com/n8n-io/n8n/blob/master/.github/pull_request_title_conventions.md ))
- [ ] [Docs updated](https://github.com/n8n-io/n8n-docs ) or follow-up
ticket created.
- [x] Tests included.
> A bug is not considered fixed, unless a test is added to prevent it
from happening again. A feature is not complete without tests.
>
> *(internal)* You can use Slack commands to trigger [e2e
tests](https://www.notion.so/n8n/How-to-use-Test-Instances-d65f49dfc51f441ea44367fb6f67eb0a?pvs=4#a39f9e5ba64a48b58a71d81c837e8227 )
or [deploy test
instance](https://www.notion.so/n8n/How-to-use-Test-Instances-d65f49dfc51f441ea44367fb6f67eb0a?pvs=4#f6a177d32bde4b57ae2da0b8e454bfce )
or [deploy early access version on
Cloud](https://www.notion.so/n8n/Cloudbot-3dbe779836004972b7057bc989526998?pvs=4#fef2d36ab02247e1a0f65a74f6fb534e ).
2023-12-07 16:22:41 +01:00