Skip to content

test(federation): Add a test to change call notification settings in … - #15511

Merged
nickvergessen merged 1 commit into
mainfrom
test/15510/allow-change-call-notification-in-fed-rooms
Jul 15, 2025
Merged

nickvergessen merged 1 commit into
mainfrom
test/15510/allow-change-call-notification-in-fed-rooms

Conversation

@nickvergessen

@nickvergessen nickvergessen commented Jul 15, 2025

Copy link
Copy Markdown
Member

…federated conversations

☑️ Resolves

🛠️ API Checklist

diff --git a/lib/Controller/RoomController.php b/lib/Controller/RoomController.php
index e08a59772..f77527814 100644
--- a/lib/Controller/RoomController.php
+++ b/lib/Controller/RoomController.php
@@ -860,6 +860,7 @@ class RoomController extends AEnvironmentAwareOCSController {
         * 200: Call notification level updated successfully
         * 400: Updating call notification level is not possible
         */
+       #[FederationSupported]
        #[NoAdminRequired]
        #[RequireLoggedInParticipant]
        public function setNotificationCalls(int $level): DataResponse {

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not possible
  • 📘 API documentation in docs/ has been updated or is not required
  • 🔖 Capability is added or not needed

…federated conversations

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen added this to the 🪺 Next Major (32) milestone Jul 15, 2025
@nickvergessen nickvergessen self-assigned this Jul 15, 2025
@nickvergessen nickvergessen added 3. to review feature: settings ⚙️ Settings and config related issues feature: api 🛠️ OCS API for conversations, chats and participants feature: call 📹 Voice and video calls feature: federation 🌐 labels Jul 15, 2025
@nickvergessen

Copy link
Copy Markdown
Member Author

/backport to stable31

@nickvergessen

Copy link
Copy Markdown
Member Author

/backport to stable30

@nickvergessen

Copy link
Copy Markdown
Member Author

Failure is expected and exactly the proof

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature: api 🛠️ OCS API for conversations, chats and participants feature: call 📹 Voice and video calls feature: federation 🌐 feature: settings ⚙️ Settings and config related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Can not change call-notification level on federated conversations

2 participants