feat: create example event when a user logs in for the first time

[st3iny]

Resolves #6608

Create an example event when a user first logs in. Admins can optionally overwrite the event with a custom ICS file.

How to test?

1. Check out this branch.
2. Create a new user.
3. Log in as the user for the first time.
4. Check the Calendar dashboard widget -> should contain a single welcome event in 7 days
5. Check the Calendar app -> should contain a single welcome event in 7 days

Optional: Upload a custom event

1. Go to Admin Settings -> Groupware
2. Upload some custom event, e.g. custom-event.ics.txt.
3. Repeat the steps above and check out your custom welcome event.

Screenshots

Admin settings (groupware section)

Upload modal

TODO

• Make sure that the personal (default) calendar is available
• Tests
• Import Sabre from 3rdparty (follow-up for Nextcloud 32: Extend OCP API for Nextcloud 32 and migrate to it)

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 85 lines in your changes missing coverage. Please review.

Project coverage is 22.93%. Comparing base (cb7d9aa) to head (1b7de04).
Report is 4 commits behind head on main.

Files with missing lines	Patch %	Lines
src/views/GroupwareAdminSettings.vue	0.00%	69 Missing ⚠️
src/services/exampleEventService.js	0.00%	9 Missing ⚠️
src/settingsAdminGroupware.js	0.00%	7 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #6648      +/-   ##
============================================
- Coverage     23.00%   22.93%   -0.07%     
- Complexity      475      518      +43     
============================================
  Files           252      260       +8     
  Lines         12110    12347     +237     
  Branches       2310     2322      +12     
============================================
+ Hits           2786     2832      +46     
- Misses         8997     9188     +191     
  Partials        327      327              

Flag	Coverage Δ
javascript	14.39% <0.00%> (-0.15%)	⬇️
php	57.82% <ø> (-1.48%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[SebastianKrupinski]

Houston, we got a problem: The personal (default) calendar is not yet created when the first login event is fired. The method IManager::getCalendarsForPrincipal() returns an empty array and thus the example event can't actually be created anywhere.

This is an easy fix. Just do a PR in the DAV app to listen for the same event and create the calendar.

[st3iny]

Houston, we got a problem: The personal (default) calendar is not yet created when the first login event is fired. The method IManager::getCalendarsForPrincipal() returns an empty array and thus the example event can't actually be created anywhere.

This is an easy fix. Just do a PR in the DAV app to listen for the same event and create the calendar.

This is actually an ancient bug in server. See my fix at: nextcloud/server#50369

[SebastianKrupinski]

Hey,

I like the approach of listening to the FirstLoginEvent, but I would do the implementation a bit different.

A. I would put the example creation code, inside the DAV app, if you think of it from a point of view, where you can have multiple calendar providers, each provider should create their own sample or use a common example, instead of only creating one example in the first calendar that might not even be the system dav calendar. Also, what if someone does not want to have the calendar web UI installed but still wants example content to show up in CalDAV. Putting the code in the CalDAV, also fixes your other issue of the calendar not existing before you add the example content.

B. I would leave the front end UI part of this in the calendar app, that way someone can upload a custom example if they have the UI installed, and the calendar providers can check if a custom event is set. But i feel like this should be part of a general defaults core app.

Another thought, we should probably implement localization of default and custom events, for multi national use, in instances like out that have people from all over the world working on the same instance.

[st3iny]

A. I would put the example creation code, inside the DAV app, if you think of it from a point of view, where you can have multiple calendar providers, each provider should create their own sample or use a common example, instead of only creating one example in the first calendar that might not even be the system dav calendar. Also, what if someone does not want to have the calendar web UI installed but still wants example content to show up in CalDAV. Putting the code in the CalDAV, also fixes your other issue of the calendar not existing before you add the example content.

Good point. The uid replacement would also be less hacky inside serve due to the availability of Sabre. I also discussed this with Christoph and Hamza and it might be a good idea to move the code to server and integrate it into Hamza's structure at nextcloud/server#50156.

Translation is not a requirement as discussed with designers and Christoph. Admins can very well upload their own events in case they want to have a custom language or simply disable the feature. However, given that the event is built inside PHP code localization could be done quite easily.

Whether the event should show up even without the Calendar app should be discussed with designers. As it stand right now, this should not be the case, so disabling Calendar will also disable the example event.

B. I would leave the front end UI part of this in the calendar app, that way someone can upload a custom example if they have the UI installed, and the calendar providers can check if a custom event is set. But i feel like this should be part of a general defaults core app.

I disagree about leaving it in Calendar. The settings UI is not really dependent on an app and should be moved where the backend lives to reduce complexity. Either way, I implemented a UI in the admin settings where admins can upload arbitrary ICS events.

[SebastianKrupinski]

Good point. The uid replacement would also be less hacky inside serve due to the availability of Sabre. I also discussed this with Christoph and Hamza and it might be a good idea to move the code to server and integrate it into Hamza's structure at nextcloud/server#50156.

I agree.

Translation is not a requirement as discussed with designers and Christoph. Admins can very well upload their own events in case they want to have a custom language or simply disable the feature. However, given that the event is built inside PHP code localization could be done quite easily.

Is it not requirement because no one thought of it? 😆

I disagree about leaving it in Calendar. The settings UI is not really dependent on an app and should be moved where the backend lives to reduce complexity. Either way, I implemented a UI in the admin settings where admins can upload arbitrary ICS events.

True.

[st3iny]

Is it not requirement because no one thought of it? 😆

Shhh 😆 No for real, the original idea was to use a static ICS file so translation was not possible technically. (At least not with our current setup.)