Skip to content

Conversation

szotsaki
Copy link

Extended the documentation with smaller fixes and added some examples that were hard to come by.

Additionally, added caveats where the implementation doesn't support something in order for other users not to debug it again.

specialized match!

Currently, `*` doesn't match INBOX.
Copy link
Author

Choose a reason for hiding this comment

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

saving via Virtual/All folder.

::: warning
Currently, imapsieve scripts that are defined within a virtual mailbox are not being called.
Copy link
Author

Choose a reason for hiding this comment

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

Comment on lines +359 to +376
group @namespaces-virtual english {
namespace virtual {
prefix = virtual/

mail_driver = virtual
mail_path = /var/lib/dovecot/virtual/en
mail_index_path = %{home}/index/virtual/en

mailbox All {
auto = no
special_use = \All
}
mailbox Flagged {
auto = no
special_use = \Flagged
}
}
}
Copy link
Author

Choose a reason for hiding this comment

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

```

::: warning
Currently, indices are not created for namespaces that are defined within groups.
Copy link
Author

Choose a reason for hiding this comment

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

@@ -108,6 +108,9 @@ This plugin registers the `imapsieve` extension with the Sieve
interpreter. This extension is enabled implicitly, which means that it
does not need to be added to the [[setting,sieve_extensions]] setting.

Additional information about using the plugin with virtual mailboxes can be
found at [[link,imapsieve-filters,virtual mailbox plugin]].
Copy link
Contributor

Choose a reason for hiding this comment

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

[vitepress] Dovecot link missing: imapsieve-filters

Copy link
Author

Choose a reason for hiding this comment

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

I'm not really familiar with Vitepress builds. I'd like to link to IMAPSieve Filters page. What would be the correct link in this case?

Copy link
Contributor

Choose a reason for hiding this comment

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

you need to add a link anchor there first, see other pages for examples

Copy link
Author

Choose a reason for hiding this comment

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

I modified the source. I don't see the CI failing with its check but github interface is bit ambiguous when these checks should run.

Comment on lines +354 to +356
Within one group it's not possible to enumerate different kind of settings, like
`prefix`, `separator`, and multiple `mailboxes`. For these cases, include the
hierarchy one above, like `namespace`. For example:
Copy link
Author

Choose a reason for hiding this comment

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

I definitely saw that namespace elements within a group (like prefix, separator, etc., except for mailbox) are not allowed.

Is my generalisation correct that mixing elements is not supported, or is it supported but some elements are excluded from this support?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants