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

[Bug] Trash items included in e-mail notification using a bookmark #1323

Closed
davidclemens opened this issue Jan 28, 2022 · 2 comments · Fixed by #1329
Closed

[Bug] Trash items included in e-mail notification using a bookmark #1323

davidclemens opened this issue Jan 28, 2022 · 2 comments · Fixed by #1329
Labels
bug Something isn't working or in unexpected ways restserver Affects the rest server.

Comments

@davidclemens
Copy link

davidclemens commented Jan 28, 2022

I keep receiving e-mail notifications including a document that has been deleted but still resides in the trash bin.

Details:
I have the following bookmark (thanks for that great new feature!) setup:

(| f:eingereicht_beihilfe=false f:eingereicht_kv=false f:eingereicht=false )

I use this bookmark in a periodic e-mail notification.
There is 1 document that has the custom field eingereicht_kv set to false. The document has, however, been deleted but is still in the trash.
This document is still included in the e-mail notifications that I receive daily, even though it shouldn't be. But if I enter the query mentioned above manually in the power-user search bar, the document does not show up (as expected).

I honestly have no clue, why this happens and I'm not sure what other information is needed to find the root cause.

Here is the job executor log for the mail task:

2022-01-28T8:00:11: Loaded bookmark '<redacted>': Some(Right((| f:eingereicht_beihilfe="false" f:eingereicht_kv="false" f:eingereicht="false" )))
2022-01-28T8:00:11: Using bookmark: Right((| f:eingereicht_beihilfe="false" f:eingereicht_kv="false" f:eingereicht="false" ))
2022-01-28T8:00:11: Attempting to send notification mail: Email(MailConfig(smtp://<redacted_smtp_dns>:587, <redacted_e-mail_1>, ***, StartTLS, 10 seconds),MailAddress(None,<redacted_e-mail_1>),NonEmptyList(MailAddress(None,<redacted_e-mail_1>), MailAddress(None,redacted_e-mail_2)))
2022-01-28T8:00:11: Send notification mail <215218450.5.1643356811548@6c3acebf41f3>
2022-01-28T8:00:11: Job execution successful

Any advice would be appreciated.

Docspell 0.31.0 on Docker

@eikek
Copy link
Owner

eikek commented Jan 29, 2022

Thank you for reporting this! This is a bug probably existing for quite some time - never noticed!

@eikek eikek added bug Something isn't working or in unexpected ways restserver Affects the rest server. labels Jan 29, 2022
@eikek eikek added this to the Docspell 0.32.0 milestone Jan 29, 2022
eikek added a commit that referenced this issue Jan 29, 2022
It didn't specify to only search in valid items

Fixes: #1323
@mergify mergify bot closed this as completed in #1329 Jan 29, 2022
@davidclemens
Copy link
Author

Thanks a lot.

Maybe that could be yet another parameter for the query language: trash=true/false.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working or in unexpected ways restserver Affects the rest server.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants