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): Auto-add LLM chain for new LLM nodes on empty canvas #10245

Conversation

OlegIvaniv
Copy link
Contributor

@OlegIvaniv OlegIvaniv commented Jul 30, 2024

Summary

  • Do not hide LLM AI nodes if there's no AI nodes on the canvas
  • If an AI node is added and there's no AI node on the canvas, auto-add LLM Chain and Chat trigger nodes
CleanShot.2024-07-30.at.14.59.34.mp4

Related Linear tickets, Github issues, and Community forum posts

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

Copy link

cypress bot commented Jul 30, 2024



Test summary

394 0 0 0Flakiness 0


Run details

Project n8n
Status Passed
Commit 9f035d6
Started Aug 5, 2024 11:36 AM
Ended Aug 5, 2024 11:41 AM
Duration 04:42 💡
OS Linux Debian -
Browser Electron 118

View run in Cypress Cloud ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Cloud

@n8n-assistant n8n-assistant bot added n8n team Authored by the n8n team ui Enhancement in /editor-ui or /design-system labels Jul 30, 2024
burivuhster
burivuhster previously approved these changes Aug 2, 2024
Copy link
Contributor

github-actions bot commented Aug 2, 2024

✅ All Cypress E2E specs passed

Copy link
Contributor

github-actions bot commented Aug 5, 2024

✅ All Cypress E2E specs passed

@jeanpaul jeanpaul merged commit 06419d9 into master Aug 5, 2024
28 checks passed
@jeanpaul jeanpaul deleted the ai-226-when-adding-an-llm-model-to-the-canvas-without-a-root-node branch August 5, 2024 11:59
MiloradFilipovic added a commit that referenced this pull request Aug 5, 2024
* master:
  feat(editor): Auto-add LLM chain for new LLM nodes on empty canvas (#10245)
  fix(core): Fix user telemetry bugs (#10293)
  refactor(core): Remove stray log from telemetry event relay (no-changelog) (#10295)
  refactor(core): Decouple user events from internal hooks (no-changelog) (#10292)
  feat(core): Support community packages in scaling-mode  (#10228)
  test(core): Move unit tests closer to testable components (no-changelog) (#10287)
  fix(core): Webhook and form baseUrl missing (#10290)
  refactor(core): Port cache config (no-changelog) (#10286)
@github-actions github-actions bot mentioned this pull request Aug 7, 2024
@janober
Copy link
Member

janober commented Aug 7, 2024

Got released with n8n@1.54.0

# 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.

4 participants