Skip to content

[_] Internxt Meet v1.5 - #282

Merged
TamaraFinogina merged 12 commits into
mainfrom
test_video_fix
Aug 31, 2026
Merged

[_] Internxt Meet v1.5#282
TamaraFinogina merged 12 commits into
mainfrom
test_video_fix

Conversation

@TamaraFinogina

@TamaraFinogina TamaraFinogina commented Aug 25, 2026

Copy link
Copy Markdown

Description

New Meet v1.5 release:

What was fixed:

[Debt] PB-6387 - update onnxruntime-web dependency
[Fix] PB-6382 - Other participants get blurry when user is main speaker
[Fix] PB-6443 - Button covers user name
[Fix] PB-6383 - Permission popup appears again after giving permissions
[Fix] PB-6444 - Room is not always closed after moderator leaves
[Fix] PB-6485 - Text dissapears when selected
[Fix] PB-6546 - Second video appears on top of the existing one
[Fix] PB-6647 - When second video appears, first one is frozen
[Feature] PB-6195 - Display 6 emojis to guarantee E2E correctness
[Feature] increase video quality and improve layout
[Maintenance] bring updates from jitsi repo for lib-meet & update dependencies

Related Issues

PB-6387
PB-6382
PB-6443
PB-6383
PB-6444
PB-6485
PB-6546
PB-6647
PB-6195

Related Pull Requests

Checklist

  • Changes have been tested locally.
  • Unit tests have been written or updated as necessary.
  • The code adheres to the repository's coding standards.
  • Relevant documentation has been added or updated.
  • No new warnings or errors have been introduced.
  • SonarCloud issues have been reviewed and addressed.
  • QA Passed

How Has This Been Tested?

QA

Additional Notes

Now 6 emojis are shown. It will help to discard cryptographic problems (if emojis are identical, crypto is fine)

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 25, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
meet-web bb6b7bc Commit Preview URL

Branch Preview URL
Aug 31 2026, 02:25 PM

@TamaraFinogina TamaraFinogina self-assigned this Aug 25, 2026
@TamaraFinogina TamaraFinogina changed the title [_] switch to lib-meet 1.0.3 [_] Internxt Meet v1.5 Aug 27, 2026
@TamaraFinogina
TamaraFinogina marked this pull request as ready for review August 31, 2026 07:29
if (ConfigService.instance.isDevelopment()) {
dispatch(openDialog('ParticipantVerificationDialog', ParticipantVerificationSASDialog, { sas }));
}
dispatch(openDialog('ParticipantVerificationDialog', ParticipantVerificationSASDialog, { sas }));

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the check has to appear in prod?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. That will be the best way for you guys to rule out crypto problems. If it gives problems, disable it again. But it should work just fine

Comment thread webpack.config.js
return "/index.html";
}

if (tpath === "/config.js" || tpath === "/interface_config.js") {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why this?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Otherwise, it was using the Jitsi config in make dev (because the path to our config was defined for prod only), and I couldn't see the difference untill changes were pushed to remote

Comment on lines +70 to +77
if (height > 0 && tileWidth > 0 && height * dpr > maxSeen.current) {
maxSeen.current = height * dpr;
dispatch(setMeasuredTileViewThumbnailSize(
Math.round(height * dpr),
Math.round(tileWidth * dpr)
));
}
};

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don’t quite understand this. Does maxSeen simply store the maximum tile size seen throughout the entire call and never decrease? Does this mean the resolution never decreases, or what effect does it have? Because it always just stores the largest value

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jitsi reduced the video quality after a few minutes, despite all other changes. It has some smart way of figuring out what quality to use, which is not working well with our customization.

I can try to remove maxSeen and check again, but paradoxically, it was the only way to stop Jitsi from reducing the video quality

@TamaraFinogina TamaraFinogina Aug 31, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@CandelR I tried removing maxSeen and resetting it to 0 in useEffect, but both resulted in quality degradation after a few minutes of the call. maxSeen blocks this degradation, but we really need to figure out how Jitsi decides video quality (highly non-trivial btw).

P.S. Video bridge ignores this and sends what it can. I think this whole quality calculation is more of an artificial limitation Jitsi sets to reduce load on the video bridge. Since we only have 10 participants (and not 100 as they support), I think we can ignore this optimization

@github-actions

Copy link
Copy Markdown

Deploying meet-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: bb6b7bc
Status:  ✅  Deploy successful!
Preview URL: https://dc504710.meet-web.pages.dev
Updated (Europe/Madrid): 31/8/2026, 16:21:23

View logs

@TamaraFinogina
TamaraFinogina merged commit ab70e0a into main Aug 31, 2026
5 of 6 checks passed
@TamaraFinogina
TamaraFinogina deleted the test_video_fix branch August 31, 2026 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants