Skip to content

only show to-device transport label in dev mode #3221

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

Merged
merged 2 commits into from
May 13, 2025

Conversation

BillCarsonFr
Copy link
Member

Draft because based on #3208

image

Copy link
Contributor

@toger5 toger5 left a comment

Choose a reason for hiding this comment

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

Looks sane.
The test might fail because its not merging into main.

This could also get a test. But we will probably add a test in the parent PR and then only adjust the test here?

@toger5 toger5 force-pushed the toger5/to-device-info-label branch 5 times, most recently from fba319e to bf576ca Compare May 13, 2025 19:41
@toger5
Copy link
Contributor

toger5 commented May 13, 2025

I hope its fine If i take over this PR since it is a bit of rebase work I hope you were not eager to do?
(and the tests)

If not just force push your version from your local branch.

@toger5 toger5 force-pushed the valere/only_show_todevice_label_in_dev_mode branch from b157a51 to 6b3f704 Compare May 13, 2025 19:47
@toger5 toger5 changed the base branch from toger5/to-device-info-label to livekit May 13, 2025 20:07
@toger5 toger5 force-pushed the valere/only_show_todevice_label_in_dev_mode branch from 7aa9794 to f0463da Compare May 13, 2025 20:11
@toger5 toger5 marked this pull request as ready for review May 13, 2025 20:20
@toger5 toger5 requested a review from a team as a code owner May 13, 2025 20:20
@toger5 toger5 requested a review from robintown May 13, 2025 20:20
Copy link
Contributor

@toger5 toger5 left a comment

Choose a reason for hiding this comment

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

@BillCarsonFr I wanted to get this off the table today so I have space for different things tomorrow.
It is basically still exactly your change.
Maybe you can still give it a quick scan and ping me tomorrow if there is any concern.

@toger5 toger5 merged commit 1ff2fb3 into livekit May 13, 2025
18 checks passed
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