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

fix(editor): Disable fromAI button for vector stores #13125

Merged
merged 4 commits into from
Feb 10, 2025
Merged

Conversation

CharlieKolb
Copy link
Contributor

Summary

Missed this one in the intial list.

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/ADO-3203/remove-vector-store-name-fromai-button

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

codecov bot commented Feb 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

@CharlieKolb CharlieKolb changed the title fix(editor): Remove fromAI button for vector store tool name fix(editor): Disable fromAI button for vector stores Feb 7, 2025
@CharlieKolb CharlieKolb marked this pull request as ready for review February 7, 2025 11:51
@n8n-assistant n8n-assistant bot added n8n team Authored by the n8n team ui Enhancement in /editor-ui or /design-system labels Feb 7, 2025
@CharlieKolb CharlieKolb added the release/backport Changes that need to be backported to older releases. label Feb 7, 2025
if (
!codex?.categories?.includes('AI') ||
!codex?.subcategories?.AI?.includes('Tools') ||
codex?.subcategories?.AI?.includes('Vector Stores') // vector stores do not support fromAI
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we add unit tests for these changes? Thanks

@CharlieKolb CharlieKolb requested a review from a team February 10, 2025 08:33
Copy link

cypress bot commented Feb 10, 2025

n8n    Run #9203

Run Properties:  status check passed Passed #9203  •  git commit cc7c9e44d9: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 CharlieKolb 🗃️ e2e/*
Project n8n
Branch Review ADO-3203
Run status status check passed Passed #9203
Run duration 04m 35s
Commit git commit cc7c9e44d9: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 CharlieKolb 🗃️ e2e/*
Committer Charlie Kolb
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 5
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 434
View all changes introduced in this branch ↗︎

Copy link
Contributor

✅ All Cypress E2E specs passed

@CharlieKolb CharlieKolb merged commit bde8420 into master Feb 10, 2025
37 checks passed
@CharlieKolb CharlieKolb deleted the ADO-3203 branch February 10, 2025 08:51
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
n8n team Authored by the n8n team release/backport Changes that need to be backported to older releases. ui Enhancement in /editor-ui or /design-system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants