-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
allow grouping of activity settings #22117
Conversation
Signed-off-by: Robin Appelman <[email protected]>
f9fc3d1
to
227e362
Compare
🤖 beep boop beep 🤖 Here are the logs for the failed build: Status of 31420: failuremysql8.0-php7.2Show full log
acceptance-app-files
Show full log
|
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.
Tested and works 👍
That's how it looks like: |
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.
Regarding the wording, the spec says this:
- Specific settings section "Notifications", not called "Activity" (All the settings go to Notifications, Activity has none as there you can filter in the interface)
- Each setting has checkmarks for "Mail" and "Push" – and you can of course also leave both unchecked
Any specific reason why the wording is not like that?
Adjusted the wording (change is in the activity repo), not that the wording/merging/etc of the individual items are not in the scope of the PR and will be changed separately |
regression:
|
Allow apps to group their activity settings, includes a group for "Files" as example
settings frontend here: nextcloud/activity#477