Skip to content

fix(CalDav): use old event information if new is missing #52882

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.

Sign up for GitHub

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

Jump to bottom
Merged
SebastianKrupinski merged 1 commit into from
Oct 28, 2025
Merged

fix(CalDav): use old event information if new is missing #52882

SebastianKrupinski merged 1 commit into from
Oct 28, 2025

Conversation

@SebastianKrupinski
Copy link
Contributor

@SebastianKrupinski SebastianKrupinski commented May 15, 2025 •
edited
Loading

Issue

Resolution

  • added check to use old event information, when a user removes all attendees the new information is blank

Checklist

@SebastianKrupinski
fix: use old event information if new is missing
dcf3d6e 
Signed-off-by: SebastianKrupinski <krupinskis05@gmail.com>
@SebastianKrupinski SebastianKrupinski self-assigned this May 15, 2025
@SebastianKrupinski SebastianKrupinski added the 3. to review Waiting for reviews label May 15, 2025
@SebastianKrupinski SebastianKrupinski marked this pull request as draft May 15, 2025 16:22
@github-project-automation github-project-automation bot moved this to 🏗️ In progress in 💌 📅 👥 Groupware team May 15, 2025
@SebastianKrupinski SebastianKrupinski mentioned this pull request May 15, 2025
Closed
@SebastianKrupinski SebastianKrupinski changed the title fix: use old event information if new is missing fix(CalDav): use old event information if new is missing May 15, 2025
@SebastianKrupinski SebastianKrupinski marked this pull request as ready for review May 20, 2025 22:09
@ChristophWurst
Copy link
Member

ChristophWurst commented Aug 4, 2025

@SebastianKrupinski have a look at the open todos in the PR description, tick off what has been done, and deliver the rest

@SebastianKrupinski SebastianKrupinski merged commit 59c9aad into master Oct 28, 2025
200 of 211 checks passed
@github-project-automation github-project-automation bot moved this from 🏗️ In progress to ☑️ Done in 💌 📅 👥 Groupware team Oct 28, 2025
@SebastianKrupinski SebastianKrupinski deleted the fix/issue-6838-use-old-event-information-when-new-is-missing branch October 28, 2025 12:38
@SebastianKrupinski
Copy link
Contributor Author

SebastianKrupinski commented Oct 28, 2025

/backport to stable32

@backportbot backportbot bot mentioned this pull request Oct 28, 2025
Merged
@SebastianKrupinski
Copy link
Contributor Author

SebastianKrupinski commented Oct 28, 2025

/backport to stable31

@backportbot backportbot bot mentioned this pull request Oct 28, 2025
Merged
@SebastianKrupinski SebastianKrupinski mentioned this pull request Dec 1, 2025
Open
@nextcloud-bot nextcloud-bot mentioned this pull request Jan 6, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ChristophWurst ChristophWurst ChristophWurst approved these changes

@GVodyanov GVodyanov GVodyanov approved these changes

@st3iny st3iny Awaiting requested review from st3iny

@tcitworld tcitworld Awaiting requested review from tcitworld tcitworld is a code owner

@kesselb kesselb Awaiting requested review from kesselb

@GretaD GretaD Awaiting requested review from GretaD

@hamza221 hamza221 Awaiting requested review from hamza221

Assignees

@SebastianKrupinski SebastianKrupinski

Labels

3. to review Waiting for reviews

Projects

💌 📅 👥 Groupware team
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Error when removing invitees from event

4 participants

@SebastianKrupinski @ChristophWurst @GVodyanov