-
Notifications
You must be signed in to change notification settings - Fork 240
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
feat: add availability action to the contacts menu #6502
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Richard Steinmetz <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6502 +/- ##
============================================
- Coverage 23.46% 23.16% -0.31%
- Complexity 453 456 +3
============================================
Files 247 250 +3
Lines 11728 11864 +136
Branches 2212 2255 +43
============================================
- Hits 2752 2748 -4
- Misses 8656 8792 +136
- Partials 320 324 +4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
So far, so good 🙏
} | ||
|
||
private function addContactsMenuScript(ContainerInterface $container): void { | ||
// ServerVersion was added in 31, but we don't care about older versions anyway |
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.
Psalm is not happy, tho
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.
But I already added a try catch around the container query 😨
I'll add it to the baseline for Nextcloud stable30
...
EDIT: Done
Part of nextcloud/server#44776
Requires nextcloud/server#49375 and nextcloud-libraries/nextcloud-vue#6238
Add a new action to the contacts menu to show the user's availability. The plan is to reuse the free-busy modal for this.
TODO
Screencasts
Avatar menu integration (from Talk)
2024-11-27.14-33-53.mp4
Contacts Menu integration (everywhere)
2024-11-27.14-35-29.mp4