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

Feature request: Tags Popup integration to trigger quickFilters assistant #224

Open
121sim opened this issue Nov 17, 2023 · 4 comments
Open
Assignees

Comments

@121sim
Copy link

121sim commented Nov 17, 2023

Hi,

I was wondering what triggers the option : "Run filters automatically when tags are change" as it does work with a right click on the message / assign a tag /, but it doesn't seem to work when using the "Tag Popup" addon (by [H.Ogi] user) by clicking on the header message (see picture below).

image

I guess it's related to the Tag Popup addon and I will ask the author too, tho it seems to use the same tags system as Quickfilters (all labels are available at the same place, and when using the right click method any label works).

Thanks.

@RealRaven2000
Copy link
Owner

So the problem is - I cannot use a simple tag listener (a routine that checks whether a tag was changed) to trigger the assistant, because filters can change tags too. And we do not want a filter to cause the assistant to create a new filter. It's same as with moving emails: I tried to change the detection of the event of moving them to a folder listener but it was a desaster, because there is no easy way to discern whether a user has moved the email or whether it was done by a filter action.

So any user interaction that can trigger the assistant by changing a tag or the location of an email must be handcrafted to the specific method. Personally, I am using tag popup too but via the unified toolbar button, mainly to manage licensing emails:

image

There is no easy way to do this via the standard API, instead there would be 2 possible paths
1 - monkey patch the existing functions from the other Add-on - this is the quick and dirty way but may potentially be unstable, because to monkey patch a function it has to exist as an experiment and has to be loaded before the quickFilters Add-on is loaded. The load order of Add-ons is never guaranteed
2 - we could try and collaborate with the author of the Add-on and make a notification method in quickFilters that could be called or registered on his side. I have done collaborations like this before with the Add-ons PrintingTools NG and Cardbook, both authors of which regularly join the weekly Add-on developer Zoom meeting.

@RealRaven2000 RealRaven2000 self-assigned this Nov 17, 2023
@RealRaven2000 RealRaven2000 changed the title Tags trigger Feature request: Tags Popup integration to trigger quickFilters assistant Nov 17, 2023
@121sim
Copy link
Author

121sim commented Nov 17, 2023

Thanks for your detailed reply.

@RealRaven2000
Copy link
Owner

Thanks for your detailed reply.

If you open up an issue with the author of Tag popup, can you link it here please? (assuming he also has a public tracking system).

@121sim
Copy link
Author

121sim commented Nov 17, 2023

Sure : https://hogi.sakura.ne.jp/hiki/?Ticket-438

@RealRaven2000 RealRaven2000 pinned this issue Apr 12, 2024
@RealRaven2000 RealRaven2000 unpinned this issue Apr 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants