-
-
Notifications
You must be signed in to change notification settings - Fork 242
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Teams v2 meeting links are always sent to an external brower rather than captured #1289
Comments
Note that this breaks the workaround discussed in #675 (that I used to use). |
If you paste that link into a channel and click from it, does it work? |
No, if I paste the link into a channel, and then click it is also opened straight into the external browser. The debug log says |
Seems like they have changed the URL again, so the regex will need updating.
…On Fri, 7 Jun 2024, 16:29 pablob127, ***@***.***> wrote:
No, if I paste the link into a channel, and then click it is also opened
straight into the external browser. The debug log says [DEBUG] Requesting
to open '
https://teams.microsoft.com/v2/?meetingjoin=true#/l/meetup-join/19'
—
Reply to this email directly, view it on GitHub
<#1289 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADJHECYW55G5EIOYAKUTR3ZGHGWHAVCNFSM6AAAAABI6ZVPMCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJVGA3TAOJVGY>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
I think is this regex https://github.com/IsmaelMartinez/teams-for-linux/blob/develop/app/mainAppWindow/index.js#L209 should also be applied in https://github.com/IsmaelMartinez/teams-for-linux/blob/develop/app/mainAppWindow/index.js#L300 . If you start teams passing the meeting link as the URL, it should open the meeting.
|
That does seem to work. The UI got a bit confused when I cancelled my joining (after all the meeting link was old). |
Can you check the pre-release 1.7.1? It should be able to handle the regEx a bit better or at least now we have the option to handle it in a config option. |
It did not work. I get an error window. For some reason I cannot copy the message, so I am pasting a screenshot. |
yep, that would be me having added a bug while trying to fix this. I don't know when I will have time to look into this. Probably just use 1.7.0 (or 1.6.1) |
Hi, can you try 1.7.1 again? @jonatasrs just fixed that issue you were seeing and it might improve that area. |
I tried the new 1.7.1, and now clicking on the links (or ctrl-clicking, or right-clicking and selecting "open link") does not do anything. Could you point me again to the command-line arguments for debugging? I tried with --webDebug, and that did not show anything when trying to open the links, and I could not find where I found arguments before. |
teams-for-linux --webDebug --appLogLevels=error,info,warn,debug |
Ok, I have more precisions now:
So the regex is working, but somehow when it is true it does not lead to opening the join meeting widget. |
Is it putting the app in focus? (I wonder if it is now filtering it...)
That area of logic is fairly complex and when we change one part, another
might "react" to it.
…On Fri, 14 Jun 2024, 17:26 pablob127, ***@***.***> wrote:
Ok, I have more precisions now:
1. When opening regular (non-meeting links), those work well and are
sent to an external browser.
2. When clicking on the meeting links (either v1 or v2), nothing
happens but the debug log says:
[DEBUG] testing RegExp onNewWindow true
[DEBUG] DEBUG - captured meetup-join
So the regex is working, but somehow when it is true it does not lead to
opening the join meeting widget.
—
Reply to this email directly, view it on GitHub
<#1289 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADJHEEXQVQKBDHIUYGWR7LZHMKUJAVCNFSM6AAAAABI6ZVPMCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRYGM3DIMBSGA>
.
You are receiving this because you modified the open/close state.Message
ID: ***@***.***>
|
Somehow I lost version 1.7.1, but when I tried the focus seems to stay in the Teams-for-Linux window (or at least that's what I remember). |
I have the exact same behaviour that @pablob127 is reporting. I have some calendar entries where I manually inserted the meeting link, and I'm not able to join directly by clicking on the link. Observing the debug log I can see those lines (testing RegExp and captured meetup-join) being written when I click the link. I found one meeting link that works when I click, and that single one, instead of outputting those log lines, outputs this:
|
Poking around I found that, if the meeting details window collapses the join info (by displaying a "Show join info" link), the link works, otherwise, it doesn't. |
This is a current limitation we have. Closing this in favor of #1098 as the original issue is now solved, but the one that stays is a duplicated one. Thanks |
Teams for Linux used to (somehow) process links to join meetings so that the meeting will run inside the app.
Since the the change to Teams v2 (or at least it feels like), links to join meetings always open in an external browser, and I have not been able to find a way to join a meeting in the app.
To Reproduce
[DEBUG] DEBUG - captured meetup-join url
[DEBUG] Requesting to open 'https://teams.microsoft.com/v2/?meetingjoin=true#/l/meetup-join/19
Expected behavior
When clicking on a new-style meeting join link, I would expect the "meetingjoin" URL to be captured so it is opened inside the app, not in an external browser.
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: