Edge browser support
fmeringdal opened this issue · comments
https://github.com/fmeringdal/nettu-meet/blob/main/frontend/src/pages/EntryPage.tsx#L126
Maybe it does work on Windows (I see it's listed under a "Windows" category), but I am using Edge browser on Linux:
https://www.microsoftedgeinsider.com/en-us/download/?platform=linux
Version 92.0.884.2 (Official build) dev (64-bit)
Also, I just noticed a typo here on this line:
https://github.com/fmeringdal/nettu-meet/blob/main/frontend/src/pages/EntryPage.tsx#L147
"bowser" should be "browser" :-)
edit: nevermind on the bowser part, I see it's using a "bowser" package lol (but I'll leave it here because I like the picture)
Thanks for the details! Edge + Linux is an interesting combo 😁
I think that edge should work regardless of OS as it uses chromium. So the only thing to do is to add an entry to the supportedBrowsers variable so that it matches with the linux + edge combo.
Do you want to try to fix that? Otherwise I will fix it when I have some time to spare :)
Let me know when it's deployed/live and I'll test it out for you.
Great! It should be live now
Hm, I'm not seeing the updates yet (I made sure the cache is cleared and whatnot):
Oh looks like something is broken in the pipeline. I pushed the update manually now, and verified that that the updated version is live now.
Perfect, just confirmed it's working great in Edge Browser on Linux :-)
Awesome! 🎉 Thanks for helping out on this!