Skip to content

feat: link upcoming event to calendar app modal #53274

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
merged 1 commit into from
Jun 3, 2025
Merged

feat: link upcoming event to calendar app modal #53274

merged 1 commit into from
Jun 3, 2025

Conversation

@Antreesy
Copy link
Contributor

@Antreesy Antreesy commented Jun 2, 2025

Summary

Before:

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

After:

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

TODO

  • ...

Checklist

@Antreesy Antreesy added this to the Nextcloud 32 milestone Jun 2, 2025
@Antreesy Antreesy requested a review from miaulalala June 2, 2025 20:28
@Antreesy Antreesy self-assigned this Jun 2, 2025
@Antreesy Antreesy force-pushed the feat/noid/link-to-calendar-event branch from 84d7e41 to f1e8058 Compare June 2, 2025 20:31
@SebastianKrupinski
Copy link
Contributor

@ChristophWurst

@ChristophWurst
Copy link
Member

yes?

ChristophWurst
ChristophWurst approved these changes Jun 3, 2025
View reviewed changes
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.

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:

@Antreesy
feat: link upcoming event to calendar app modal
e2265ff 
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy Antreesy force-pushed the feat/noid/link-to-calendar-event branch from f1e8058 to e2265ff Compare June 3, 2025 07:25
@Antreesy Antreesy merged commit 6bfdcc9 into master Jun 3, 2025
202 of 206 checks passed
@Antreesy Antreesy deleted the feat/noid/link-to-calendar-event branch June 3, 2025 12:05
@Antreesy
Copy link
Contributor Author

Antreesy commented Jun 3, 2025

/backport to stable31

@backportbot backportbot bot mentioned this pull request Jun 3, 2025
Merged
@Antreesy
Copy link
Contributor Author

Antreesy commented Jun 3, 2025

/backport to stable30

@backportbot backportbot bot mentioned this pull request Jun 3, 2025
Merged
@skjnldsv skjnldsv mentioned this pull request Aug 19, 2025
Merged
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
@skjnldsv skjnldsv added this to the Nextcloud 32 milestone Oct 2, 2025
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

@miaulalala miaulalala miaulalala approved these changes

@st3iny st3iny Awaiting requested review from st3iny

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

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

Assignees

@Antreesy Antreesy

Labels

3. to review Waiting for reviews enhancement feature: dav

Projects

None yet

Milestone

Nextcloud 32

Development

Successfully merging this pull request may close these issues.

6 participants

@Antreesy @SebastianKrupinski @ChristophWurst @miaulalala @skjnldsv