Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[front] Use auto tags filtering #10510

Merged
merged 7 commits into from
Feb 5, 2025
Merged

[front] Use auto tags filtering #10510

merged 7 commits into from
Feb 5, 2025

Conversation

tdraier
Copy link
Contributor

@tdraier tdraier commented Feb 4, 2025

fixes: https://github.com/dust-tt/tasks/issues/2053
depends on #10421

Description

Adds tagsIn/tagsNot in action params. These should be added only when one datasource is in "auto" - for now in comments until #10421 is merged. Same tags are applied on all datasources where "auto" is set (check to add also once tagsMode is available)
Stores the generated values in action and displays it in the ActionDetails panel

Tests

Risk

none

Deploy Plan

Run migration, deploy front

@tdraier tdraier changed the title Thomas/auto tags [front] Use auto tags filtering Feb 4, 2025
Copy link

github-actions bot commented Feb 4, 2025

Warnings
⚠️

Files in **/lib/models/ have been modified and the PR has the migration-ack label. Don't forget to run the migration from prodbox.

Generated by 🚫 dangerJS against 8c4cca9

@tdraier tdraier requested a review from PopDaph February 5, 2025 09:30
@tdraier tdraier requested a review from PopDaph February 5, 2025 12:20
Copy link
Contributor

@PopDaph PopDaph left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@tdraier tdraier added the migration-ack 📁 Label to acknowledge that a migration is required. label Feb 5, 2025
@tdraier tdraier merged commit ae31644 into main Feb 5, 2025
11 checks passed
@tdraier tdraier deleted the thomas/auto-tags branch February 5, 2025 15:16
tdraier added a commit that referenced this pull request Feb 6, 2025
* Retrieval with auto tags

* add migration

* fixed schema
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
migration-ack 📁 Label to acknowledge that a migration is required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants