Skip to content

feat: calendar read and write federation#58008

Merged
SebastianKrupinski merged 1 commit intomasterfrom
feat/calendar-federation-readwrite
Feb 26, 2026
Merged

feat: calendar read and write federation#58008
SebastianKrupinski merged 1 commit intomasterfrom
feat/calendar-federation-readwrite

Conversation

@SebastianKrupinski
Copy link
Contributor

@SebastianKrupinski SebastianKrupinski commented Feb 3, 2026

  • Resolves: # feature request

Summary

Testing

  • Federate two instances of NC
  • Share a calendar with user from instance A to another user on instance B
  • Update a event on instance B in calendar from instance A

Checklist

@SebastianKrupinski SebastianKrupinski self-assigned this Feb 3, 2026
@SebastianKrupinski SebastianKrupinski added the 2. developing Work in progress label Feb 3, 2026
@github-project-automation github-project-automation bot moved this to 🏗️ In progress in 💌 📅 👥 Groupware team Feb 3, 2026
@SebastianKrupinski SebastianKrupinski force-pushed the feat/calendar-federation-readwrite branch from 020d2bc to 8291d3c Compare February 3, 2026 02:39
@SebastianKrupinski SebastianKrupinski force-pushed the feat/calendar-federation-readwrite branch 2 times, most recently from 04a42b2 to fb1b0a2 Compare February 3, 2026 21:43
@SebastianKrupinski SebastianKrupinski added this to the Nextcloud 33 milestone Feb 3, 2026
@SebastianKrupinski SebastianKrupinski added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Feb 3, 2026
@kesselb
Copy link
Contributor

kesselb commented Feb 4, 2026

The failing AppSettingsControllerTest is fixed by #58050

@SebastianKrupinski
Copy link
Contributor Author

AppSettingsControllerTest

Ty

@SebastianKrupinski SebastianKrupinski force-pushed the feat/calendar-federation-readwrite branch from fb1b0a2 to 565d4c0 Compare February 4, 2026 20:50
@SebastianKrupinski SebastianKrupinski marked this pull request as ready for review February 5, 2026 14:32
@SebastianKrupinski SebastianKrupinski requested review from CarlSchwan, icewind1991, leftybournes and salmart-dev and removed request for a team February 5, 2026 14:32
@SebastianKrupinski
Copy link
Contributor Author

Up for review will fix psalm in the mean time

@ChristophWurst
Copy link
Member

@SebastianKrupinski tell us how this can be tested, please

@SebastianKrupinski
Copy link
Contributor Author

@SebastianKrupinski tell us how this can be tested, please

Okay, done, updated desciption

@blizzz blizzz modified the milestones: Nextcloud 33, Nextcloud 34 Feb 16, 2026
@SebastianKrupinski SebastianKrupinski force-pushed the feat/calendar-federation-readwrite branch from 2447722 to 6fb0f9f Compare February 17, 2026 15:29
@SebastianKrupinski
Copy link
Contributor Author

@hamza221

* Quitting a share, Unsharing -> don't update for the other user 🚫  (Can be fixed in a followup )

This is done on re-sync, if the request in unauthorized, its removed

* When running sync command with one of the received shared calendars being unshared from the owner (doesn't update because of Point 2 👆🏼)

This was already being caught an handled the original implementation was re throwing the error now we log only

@hamza221
Copy link
Contributor

Notes from Yesterday's test:

  • Sharing, Editing privileges, read, Edit: work as expected ✅
  • Quitting a share, Unsharing -> don't update for the other user 🚫 (Can be fixed in a followup )
  • When running sync command with one of the received shared calendars being unshared from the owner (doesn't update because of Point 2 👆🏼)
Failed to sync calendar f98f0703147a5feeb0180ca9351b1783: Client error: `REPORT http://nextcloud.local/remote.php/dav/remote-calendars/YWRtaW5AaHR0cDovL3N0YWJsZTMzLmxvY2Fs/federated_shared_by_admin``resulted in a``401 Unauthorized` response:
<?xml version="1.0" encoding="utf-8"?>
<d:error xmlns:d="DAV:" xmlns:s="http://sabredav.org/ns">
  <s:exception>Sabre\DA (truncated...) 

Should be fixed or caught and handled

Quitting a share doesn't update for the sharer (Can be fixed in a followup as it doesn't affect any functionality )

Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

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

Code looks good

@SebastianKrupinski SebastianKrupinski force-pushed the feat/calendar-federation-readwrite branch from 7003fe2 to 07b262f Compare February 25, 2026 15:16
Copy link
Contributor Author

@SebastianKrupinski SebastianKrupinski left a comment

Choose a reason for hiding this comment

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

Done. Also added some proper typing on some of the methods

@SebastianKrupinski SebastianKrupinski force-pushed the feat/calendar-federation-readwrite branch from 07b262f to c98e4fa Compare February 25, 2026 15:27
Signed-off-by: SebastianKrupinski <krupinskis05@gmail.com>
@SebastianKrupinski
Copy link
Contributor Author

/backport to stable33

@SebastianKrupinski SebastianKrupinski merged commit ca8050b into master Feb 26, 2026
181 of 186 checks passed
@SebastianKrupinski SebastianKrupinski deleted the feat/calendar-federation-readwrite branch February 26, 2026 14:01
@github-project-automation github-project-automation bot moved this from 🏗️ In progress to ☑️ Done in 💌 📅 👥 Groupware team Feb 26, 2026
@szaimen
Copy link
Contributor

szaimen commented Feb 26, 2026

🎉🎉🎉🎉🎉

@SebastianKrupinski
Copy link
Contributor Author

🎉🎉🎉🎉🎉

I take it this is something you have been waiting for?

@szaimen
Copy link
Contributor

szaimen commented Feb 26, 2026

🎉🎉🎉🎉🎉

I take it this is something you have been waiting for?

Yeah, only like 5 years or something 😅

@SebastianKrupinski
Copy link
Contributor Author

Yeah, only like 5 years or something 😅

🐢 🐢🐢🐢🐢🐢 not 🐰

@tcitworld
Copy link
Member

Yeah, only like 5 years or something

5 years ? I have a 10 year-old WIP branch for this 😄
Congrats again for the work.

@rugk
Copy link

rugk commented Feb 26, 2026

Federate two instances of NC

I hope it also works when two instances are not explicitly federated, like when sharing files? (You can enter the full user@domain handle there.)

@SebastianKrupinski
Copy link
Contributor Author

5 years ? I have a 10 year-old WIP branch for this 😄 Congrats again for the work.

Yikes!!!

And thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: ☑️ Done

Development

Successfully merging this pull request may close these issues.

Federated calendar and contacts sharing

9 participants