feat: automated appointment creation

[SebastianKrupinski]

Summary

• Resolves: Add invites to calendar automatically (tentative without respond) #7750
• Requires: feat: automated appointment creation server#56924
• Added option in account setting to enable feature per account
• Modified imip service to use new imip handler

Testing

• Navigate to mail account settings, turn on "Automatically create tentative appointments"
• Using another external account/service send a calendar invitation to the configure nc mail account
• Trigger cron job to run background jobs

[ChristophWurst]

What about #11756?

[ChristophWurst]

Looks good so far

Is the server change a hard requirement? If so, add a version check

[SebastianKrupinski]

Is the server change a hard requirement? If so, add a version check

This only is a issue in version 31 kind of, the required method is protected, we can discuss in 1 to 1

[DerDreschner]

Tested successfully on my machine, but does work only on messages delivered to the Inbox. When creating a rule on Webmin/Virtualmin to put it in another folder, it doesn't work anymore. But I guess that's expected behaviour. 😃 Their implementation doesn't use Sieve, so, it may differ there.

Looking forward to use it on a daily basis!

[SebastianKrupinski]

Tested successfully on my machine, but does work only on messages delivered to the Inbox. When creating a rule on Webmin/Virtualmin to put it in another folder, it doesn't work anymore. But I guess that's expected behaviour. 😃 Their implementation doesn't use Sieve, so, it may differ there.

This should work on other mail boxes, we process all the mail boxes, the issue might be that the alternate mailbox has not been sync'd with NC yet, we do not sync all mailboxes all the time.

[DerDreschner]

Yep, that was indeed the issue here! When marking the mailbox as Sync in background, it works. Opening the folder with Subscribed enabled alone is not enough though.

[kesselb]

options.absent is defined as not set or "create". "ignore" is not a valid value.

[SebastianKrupinski]

Server PR is merged... will rebase in a couple hours will see what psalm says

[kesselb]

static-psalm-analysis dev-master should pass 🥳

static-psalm-analysis dev-stable32 will still fail 💥

We could change OCP\Calendar\IManager.handleIMip in stable32 like

@param array{absent?: "create"|"ignore", recipient?: string, ...} $options

To avoid the warning about "ignore" and make psalm more relax about the unknown "absentCreateStatus" with the "...".

That isn't ideal, but at least we don't have to suppress some errors or bump the baseline only for one supported version.

[SebastianKrupinski]

Yeah... I agree we need to adjust stable32 cause psalm will wine again

[SebastianKrupinski]

nextcloud/server#57582