Skip to content

feat: link upcoming event to calendar app modal - #53274

Merged
Antreesy merged 1 commit into
masterfrom
feat/noid/link-to-calendar-event
Jun 3, 2025
Merged

feat: link upcoming event to calendar app modal#53274
Antreesy merged 1 commit into
masterfrom
feat/noid/link-to-calendar-event

Conversation

@Antreesy

Copy link
Copy Markdown
Contributor

Summary

Before:

"https://nextcloud.local/index.php/apps/calendar/"

After:

"https://nextcloud.local/index.php/apps/calendar/edit/L3JlbW90ZS5waHAvZGF2L2NhbGVuZGFycy9hbnRyZWVzeS9wZXJzb25hbC90NURhdTgyT0M1anVZRU9tWVRIN2xDaE9qY3RlTjFNRi5pY3M="

TODO

  • ...

Checklist

@AntreesyAntreesy added this to the Nextcloud 32 milestone Jun 2, 2025
@Antreesy
Antreesy requested a review from miaulalalaJune 2, 2025 20:28
@AntreesyAntreesy self-assigned this Jun 2, 2025
@Antreesy
Antreesyforce-pushed the feat/noid/link-to-calendar-event branch from 84d7e41 to f1e8058CompareJune 2, 2025 20:31
@SebastianKrupinski

Copy link
Copy Markdown
Contributor

@ChristophWurst

@ChristophWurst

Copy link
Copy Markdown
Member

yes?

@ChristophWurstChristophWurst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fine by me because it builds on existing technical debt and doesn't add new.

This will work until it doesn't because routes change in Calendar

:shipit:

Comment threadapps/dav/lib/CalDAV/UpcomingEventsService.php Outdated
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy
Antreesyforce-pushed the feat/noid/link-to-calendar-event branch from f1e8058 to e2265ffCompareJune 3, 2025 07:25
@Antreesy
Antreesy merged commit 6bfdcc9 into masterJun 3, 2025
@Antreesy
Antreesy deleted the feat/noid/link-to-calendar-event branch June 3, 2025 12:05
@Antreesy

Copy link
Copy Markdown
ContributorAuthor

/backport to stable31

@Antreesy

Copy link
Copy Markdown
ContributorAuthor

/backport to stable30

@skjnldsvskjnldsv added this to the Nextcloud 32 milestone Oct 2, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@Antreesy@SebastianKrupinski@ChristophWurst@miaulalala@skjnldsv