Skip to content

Feat/noid/conversation presets andy#17169

Open
AndyScherzinger wants to merge 4 commits intomainfrom
feat/noid/conversation-presets-andy
Open

Feat/noid/conversation presets andy#17169
AndyScherzinger wants to merge 4 commits intomainfrom
feat/noid/conversation-presets-andy

Conversation

@AndyScherzinger
Copy link
Member

@AndyScherzinger AndyScherzinger commented Feb 18, 2026

For @nickvergessen

Plan 1m 42s
Exec 45s

β˜‘οΈ Resolves

  • Fix #…

πŸ–ŒοΈ UI Checklist

πŸ–ΌοΈ Screenshots / Screencasts

🏚️ Before 🏑 After
Screenshot before Screenshot after

🚧 Tasks

  • ...

🏁 Checklist

  • 🌏 Tested with different browsers / clients:
    • Chromium (Chrome / Edge / Opera / Brave)
    • Firefox
    • Safari
    • Talk Desktop
    • Integrations with Files sidebar and other apps
    • Not risky to browser differences / client
  • πŸ–ŒοΈ Design was reviewed, approved or inspired by the design team
  • ⛑️ Tests are included or not possible
  • πŸ“— User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required

πŸ› οΈ API Checklist

🚧 Tasks

  • ...

🏁 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

@nickvergessen nickvergessen changed the base branch from main to feat/noid/conversation-presets February 18, 2026 14:08
@nickvergessen nickvergessen force-pushed the feat/noid/conversation-presets branch from e8a8123 to 2c81609 Compare February 19, 2026 07:55
@AndyScherzinger AndyScherzinger force-pushed the feat/noid/conversation-presets-andy branch from 2ae0908 to 05d98f1 Compare February 19, 2026 09:35
@AndyScherzinger
Copy link
Member Author

Can't run the openAPI calls locally unfortunately

@AndyScherzinger AndyScherzinger force-pushed the feat/noid/conversation-presets-andy branch 2 times, most recently from 6eb9324 to 52e2564 Compare February 19, 2026 12:45
Base automatically changed from feat/noid/conversation-presets to main February 19, 2026 13:03
AndyScherzinger and others added 2 commits February 19, 2026 14:03
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen force-pushed the feat/noid/conversation-presets-andy branch from 52e2564 to afe870d Compare February 19, 2026 22:15
@miaulalala
Copy link
Contributor

    Given user "participant1" gets available presets with 200 (v1) # FeatureContext::userSeesTheFollowingPresets()
      | identifier   | name         | parameters                                                                                                                                               |
      | default      | default      | {"roomType":2,"readOnly":0,"listable":0,"messageExpiration":0,"lobbyState":0,"sipEnabled":0,"permissions":0,"recordingConsent":0,"mentionPermissions":0} |
      | forced       | forced       | []                                                                                                                                                       |
      | webinar      | Webinar      | {"lobbyState":1,"mentionPermissions":1,"permissions":389,"recordingConsent":1,"roomType":3}                                                              |
      | presentation | Presentation | {"mentionPermissions":1,"permissions":389,"recordingConsent":1}                                                                                          |
      | hallway      | Hallway      | {"listable":1,"messageExpiration":3600}                                                                                                                  |
      {"ocs":{"meta":{"status":"failure","statuscode":996,"message":"Internal Server Error\nCannot call abstract method OCA\\Talk\\RoomPresets\\APreset::getIdentifier() in file '\/home\/runner\/work\/spreed\/spreed\/apps\/spreed\/lib\/RoomPresets\/APreset.php' line 32"},"data":[]}}

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants