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

Problem with +tag in the title when using fetching from a mailbox #10

Closed
bepolymathe opened this issue Sep 15, 2022 · 4 comments · Fixed by #12
Closed

Problem with +tag in the title when using fetching from a mailbox #10

bepolymathe opened this issue Sep 15, 2022 · 4 comments · Fixed by #12
Assignees
Labels
enhancement New feature or request

Comments

@bepolymathe
Copy link

Hi @bishoy-magdy

After some tests (and unless I'm mistaken), it seems that tags are not correctly supported in the title when using the option to retrieve all new messages from a mailbox rather than from a particular address.

@bishoy-magdy
Copy link
Collaborator

bishoy-magdy commented Sep 15, 2022

Hi @bepolymathe

A big thanks for testing the plugin.

Honestly, this isn't an issue. I've already prevented the use of @ or # in the subject or first line in mailbox monitoring, which means it adds folders or tags by using the @ or # syntax when only monitoring a specific email account (You can only, in this case, edit the subject or first line of an email). This is because the mailbox monitoring feature is for users who don't want to use forwarding or monitor a specific e-mail, so it just fetches new or unread messages from the specified mailbox and puts them in the folder that the user-specified.

@bishoy-magdy bishoy-magdy self-assigned this Sep 15, 2022
@bepolymathe
Copy link
Author

I understand your point of view. Let me explain my use case. I have two main emails. One professional, one personal. I have a third address to which I forward the pro or personal emails that need it. For example, I would like to transfer pro mail by passing the "pro" tag.
I would find it useful... but I can live without it.

@bishoy-magdy
Copy link
Collaborator

This sounds like a great idea, I will mark it as a feature, but this feature will only work when monitoring only the inbox because that's the only place where the new message will be!

Do you think +folder support in the subject while mailbox monitoring is also useful if the user wants to map the note to more than one folder?

@bishoy-magdy bishoy-magdy added the enhancement New feature or request label Sep 17, 2022
@bepolymathe
Copy link
Author

I will mark it as a feature

Oh great !

but this feature will only work when monitoring only the inbox because that's the only place where the new message will be

That’s fine !

Do you think +folder support in the subject while mailbox monitoring is also useful if the user wants to map the note to more than one folder?

I'm not sure that's necessary...

roman-r-m added a commit that referenced this issue Oct 1, 2022
feature/Add tags mentioned in the subject to the note when monitoring a specific mailbox.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants