Skip to content
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.

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

Diff of event updates #52

Open
nickvergessen opened this issue Jan 4, 2023 · 5 comments
Open

Diff of event updates #52

nickvergessen opened this issue Jan 4, 2023 · 5 comments
Labels
enhancement New feature or request

Comments

@nickvergessen
Copy link
Member

With nextcloud/server#35743 (comment) it might be possible to improve the update notification to outline what changed (name, time, ...?)

@nickvergessen nickvergessen added the enhancement New feature or request label Jan 4, 2023
@phil-lipp
Copy link

Yes, that would be great! The notifications right now are pretty bare bones and sometimes plain useless as I cant see what another person changed. Having 5 notifications of "Person A changed event X" and trying to figure out what each individual changed is not feasible most of the time.

@nickvergessen
Copy link
Member Author

Having 5 notifications of "Person A changed event X" and trying to figure out what each individual changed is not feasible most of the time.

Well depends of course. In my home usecase the mostlikely thing to change is the time and the starting time is also visible in the notifications, so for me the most important info is there.

But yeah I will look into adding at least the list of changed properties

@nickvergessen
Copy link
Member Author

Hmm actually we don't receive the "before" data in the CalendarObjectUpdatedEvent event so we can also not diff it :(

@phil-lipp
Copy link

Hmm actually we don't receive the "before" data in the CalendarObjectUpdatedEvent event so we can also not diff it :(

Well then can't that change? Is it not possible for the server to provide that info?

@nickvergessen
Copy link
Member Author

It would be possible I guess, but required some work on multiple levels and I'm not familiar with that code base

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants