feat(meetings): selectively invite users from lobby

[Antreesy]

☑️ Resolves

• Fix Direct option to allow a user to join while the lobby is enabled #8601
• If lobby is enabled, a new button is appeare in participant action to explicitly change lobby permission for that one participant
• isInLobby was removed from chatIdentifier:
  • chat is mounted/unmounted with param change anyway, so no need to depend on it
  • it leaves outdated cancel requests in the store:
    

Important

HPB is required to be enabled, to receive signaling messages

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

In	Out

lobby-permissions.mp4

🚧 Tasks

• Check wording
• Decide on selected icons

🏁 Checklist

• 🌏 Tested with Chrome, Firefox and Safari or should not be risky to browser differences
• 🖌️ 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

[nickvergessen]

I would say it should be even more visible? Directly show it on the participant list next to the ...?

[Antreesy]

I would say it should be even more visible?

Updated code, icons (unlock/lock for account fit more IMO) and screenshots

[nickvergessen]

Not sure about the wording of the buttons. cc @jancborchardt for better ideas

[jancborchardt]

Improved wording could be:

• Move to conversation
• Move back to lobby

"Move" seems more fitting than "invite" since it is a direct action, right?

[Antreesy]

"Move" seems more fitting than "invite" since it is a direct action, right?

Agree. Maybe Move user would be more explicable, but I'm fine with short option too
Participant is already in the conversation, but can't think of other wording, so could proceed with that.

Any thoughts over icons used?

[jancborchardt]

"Move" seems more fitting than "invite" since it is a direct action, right?

Agree. Maybe Move user would be more explicable, but I'm fine with short option too
Participant is already in the conversation, but can't think of other wording, so could proceed with that.

I would use the short wording. We want to phase out the wording "user", and the text is long enough already. :)

Any thoughts over icons used?

Yep, wrote as reply in @DorraJaouad's code comment.

[nickvergessen]

We want to phase out the wording "user", and the text is long enough already. :)

Also note this is not only about "users", but also about "guests".
That's why we use the word "participant" e.g. on the remove option. Should use it here too.

[Antreesy]

Should use it here too.

Or not use it at all, to keep it short (same is done for 'Promote / demote to moderator')

[DorraJaouad]

Good to go.