Repository navigation
Conversation
|
Thanks @Koc That's definitely matching what the current teams efforts are going for. @marcoambrosini Is this what you had in mind for managing members from within an app? |
6742982 to
cada522
Compare
Signed-off-by: Kostiantyn Miakshyn <molodchick@gmail.com>
cada522 to
768faef
Compare
mejo-
left a comment
There was a problem hiding this comment.
As far as I understood it, the plan was to provide a shared members management widget that can be injected in other apps, but to keep the possibility to manage members in Collectives without having to leave the app. And that's also what I personally would prefer.
|
@mejo- I can try to do that, but need some more info:
|
|
@Koc thanks, much appreciated. I think the best would be to align with the design team (Cc @marcoambrosini), maybe even in a quick coordination call? |
Sorry for the late reply. Yes, this is exactly what I think is best. |
|
@marcoambrosini can you please answer to my questions, then?
|
📝 Summary
This is a continuation of the work started in #2518 and #2443 to optimize the members list in Collectives.
nextcloud/circlesv35 now provides its own members management UI, so Collectives no longer needs to duplicate that functionality. This PR detects that capability and, when it is available, turns theMembersWidgetmembers button into a link that opens the Circles team settings in a new tab. For older Circles versions the existing member management popup is preserved unchanged.Benefits
fullDetails=trueis skipped when the Circles app already provides the management UI.🖼️ Screenshots
❓ Open question
🚧 TODO
🏁 Checklist
npm run lint/npm run stylelint/composer run cs:check)🤖 AI (if applicable)