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

Add imip processing in the background #6812

Merged
merged 1 commit into from
Sep 9, 2022

Conversation

miaulalala
Copy link
Contributor

@miaulalala miaulalala commented Jun 30, 2022

Fixes #6802

Questions:

what happens when I have a very large mailbox where I never trigger the preview for messages below the fold?

@miaulalala miaulalala added enhancement 2. developing feature:imip For all tasks related to the iMIP functionality labels Jun 30, 2022
@miaulalala miaulalala self-assigned this Jun 30, 2022
lib/Db/Message.php Outdated Show resolved Hide resolved
lib/Db/Message.php Outdated Show resolved Hide resolved
lib/Db/MessageMapper.php Outdated Show resolved Hide resolved
lib/Migration/Version1140Date20220630113356.php Outdated Show resolved Hide resolved
lib/Migration/Version1140Date20220630113356.php Outdated Show resolved Hide resolved
lib/Migration/Version1140Date20220630113356.php Outdated Show resolved Hide resolved
@miaulalala miaulalala force-pushed the enh/process-imip-in-background branch from b43b7fe to f113610 Compare July 19, 2022 12:01
@miaulalala miaulalala force-pushed the enh/process-imip-in-background branch from 1bfa02a to 649689e Compare July 29, 2022 14:18
@miaulalala

This comment was marked as off-topic.

@kevincox

This comment was marked as off-topic.

@miaulalala

This comment was marked as off-topic.

@kevincox

This comment was marked as off-topic.

@miaulalala

This comment was marked as off-topic.

@miaulalala miaulalala force-pushed the enh/process-imip-in-background branch from 649689e to fa7ec4e Compare August 11, 2022 10:44
@miaulalala miaulalala marked this pull request as ready for review August 11, 2022 12:14
@miaulalala miaulalala force-pushed the enh/process-imip-in-background branch from fa7ec4e to 768221d Compare August 11, 2022 12:24
@miaulalala
Copy link
Contributor Author

miaulalala commented Aug 11, 2022

  • add another BG job that processes all messages until 2 weeks into the past for structure Analysis. (should make mail app faster)

Copy link
Contributor

@kesselb kesselb left a comment

Choose a reason for hiding this comment

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

Don't forget to bump the version number to install the background job.

tests/Unit/Service/PreprocessingServiceTest.php Outdated Show resolved Hide resolved
lib/Db/MessageMapper.php Outdated Show resolved Hide resolved
lib/IMAP/MessageMapper.php Show resolved Hide resolved
lib/IMAP/MessageMapper.php Show resolved Hide resolved
lib/Service/IMipService.php Outdated Show resolved Hide resolved
lib/Db/MessageMapper.php Outdated Show resolved Hide resolved
@miaulalala miaulalala force-pushed the enh/process-imip-in-background branch from 62463a8 to 33842a8 Compare August 18, 2022 09:26
@miaulalala
Copy link
Contributor Author

miaulalala commented Aug 22, 2022

just waiting for server PR now

@miaulalala miaulalala force-pushed the enh/process-imip-in-background branch from 49ed373 to cc930ce Compare August 22, 2022 20:14
@miaulalala
Copy link
Contributor Author

miaulalala commented Sep 6, 2022

stable22, 23 and 24 test failing is not surprising as this isn't backported

@ChristophWurst ChristophWurst added this to the v2.0.0 milestone Sep 6, 2022
@miaulalala
Copy link
Contributor Author

@ChristophWurst I added a check on the IManager to see if the method handleImipReply exists so we can avoid the whole test suite exploding for the next however many releases.

Shamelessly stolen from https://github.com/nextcloud/calendar/blob/main/tests/php/unit/Service/Appointments/BookingServiceTest.php

@ChristophWurst ChristophWurst mentioned this pull request Sep 8, 2022
6 tasks
lib/BackgroundJob/PreviewEnhancementProcessingJob.php Outdated Show resolved Hide resolved
lib/Db/MessageMapper.php Outdated Show resolved Hide resolved
lib/Migration/Version1140Date20220630113356.php Outdated Show resolved Hide resolved
lib/Service/MailManager.php Outdated Show resolved Hide resolved
lib/Service/PreprocessingService.php Outdated Show resolved Hide resolved
lib/Service/PreprocessingService.php Outdated Show resolved Hide resolved
@miaulalala miaulalala force-pushed the enh/process-imip-in-background branch 2 times, most recently from e2197b8 to 35cf81a Compare September 8, 2022 13:42
@ChristophWurst
Copy link
Member

linter

lib/BackgroundJob/IMipMessageJob.php Outdated Show resolved Hide resolved
lib/BackgroundJob/IMipMessageJob.php Outdated Show resolved Hide resolved
Signed-off-by: Anna Larch <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review enhancement feature:imip For all tasks related to the iMIP functionality
Projects
Development

Successfully merging this pull request may close these issues.

Process iMip responses
4 participants