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 logacceptance-app-files
Show full log |
|
That's how it looks like: |
There was a problem hiding this comment.
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