SrMouraSilva
Use channel name instead of random UUID for call name -> stable call name
Hi Thanks for publishing this plugin opensource! I have used similar approach (based on http://g.co/meet/) but I have implemented that as a custom slash command which triggers a URL. Your approach looks much nicer however.
I have just one suggestion - to use "stable" and human readable names for the calls instead of random uuid. In my slash command I used channel name for this, and it worked quite nicely.
The benefit of using channel name, is that this becomes a meeting "alias" in google, so can be remembered and used again.
Hi @tmotyl.
I thought that when someone clicked the button, a text input would appear asking for the desired room name. But it turned out that the UUID was enough for me.
The benefit of using channel name, is that this becomes a meeting "alias" in google, so can be remembered and used again.
This is great.
The benefit of using channel name
The use of the channel name can be problematic if there are more than one team into the Mattermost and each have the same name. Maybe a concatenation <team>-<channel> would be better.
Also, is necessary to remove punctuations, accents, spaces. Try to enter:
- https://g.co/meet/just%20a%20example
- https://g.co/meet/acentua%C3%A7%C3%A3o
- https://g.co/meet/dot.dot
Maybe apply a regex that replaces anything not in [a-zA-Z0-9] to -.
What do you think?
The channel name I'm getting from the mattermost (via POST parameter) is already URL compliant
e.g. for a channel name "test.test" the 'channel_name' param is "testtest". This name is also used as a "slug"/part of the channel URL. See the screen below.
Maybe you can get this property in the code too?
Thanks for the interesting point regarding team in the channel name. I didn't stumbled upon this issue, because slash commands are configured per team, so I can have different prefixes if needed.
Hey, @tmotyl. I migrate this repository for a new into https://github.com/SrMouraSilva/mattermost-google-meet-plugin
The new plugin now uses <team name>-<channel name>, where name is the string that is present in the URL.
Please, try to use it.
Note: In this image, the text are in portuguese because I add internationalization (only en and pt-BR currently).