-
Notifications
You must be signed in to change notification settings - Fork 244
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
Accept and decline invitations #3864
Conversation
Codecov Report
@@ Coverage Diff @@
## main #3864 +/- ##
============================================
- Coverage 29.61% 29.36% -0.26%
+ Complexity 322 318 -4
============================================
Files 220 220
Lines 7529 7455 -74
Branches 993 925 -68
============================================
- Hits 2230 2189 -41
+ Misses 5299 5266 -33
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
This makes me notice we don't have any visual feedback in the calendar grid for when an event invitation is rejected? |
Indeed. This would be a nice followup. |
fae3fc5
to
f7e4e89
Compare
I like it, but I dislike the button colours. Is there any way to make them fit in with them theming? Or maybe ask the design team for some input? |
the accept one is a primary button so its the same as the colour of the theme(see also his example is green as the theme one), the decline is the error one that we use everywhere. So if we change it here, should be changed everywhere. |
Ah, I didn't realise it's the same colour as the theme :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code looks good but i cannot test it
79f0890
to
5f39858
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works as expected. UX and design can be improved :)
Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
5f39858
to
58c4df7
Compare
The button that corresponds to the current participation status is automatically hidden. E.g., the accept button will be hidden if I already accepted earlier. |
Maybe we can just disable it and update the label ( |
This should be discussed in the next design review. I'll add it to the list. |
Fixes #144
According to https://icalendar.org/CalDAV-Scheduling-RFC-6638/b-3-example-attendee-replying-to-an-invitation.html responding to an invitation is done by setting the participation status and sending the modified ics data to the server. Thunderbird seems to do the same thing.
The local participation status in the invitees list is updated right away (without reloading the page). There is one minor side effect: The accept/decline buttons internally trigger a full save which also persists changes of e.g. reminders. Fixing this would need a lot of potentially ugly code. Again, Thunderbird also ignores this case and other changes to the event are also saved upon accepting or declining.
Accept using the sidebar editor
Decline using the simple editor
TODO