Skip to content
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

chore: Upgrade - WebRTC.Wasm Sample to run with latest uno and latest VS 2022 #153

Conversation

aesteves900
Copy link
Contributor

No description provided.

@jeromelaban
Copy link
Member

Make sure to use the name of the sample you're upgrading in the PR title. Also, it looks like we missed previously missed this sample for the upgrade (or that we missed it when it was first integrated). Good that we're updating this one now :)

@aesteves900 aesteves900 self-assigned this Sep 21, 2022
@aesteves900 aesteves900 marked this pull request as draft September 21, 2022 13:35
@aesteves900
Copy link
Contributor Author

@jeromelaban , Okay I will do one PR for each integration. Ad back to the first to do "exclusion" to the legacy folder. Despite that, I'm stuck with this error: I will time-box, update and test for two hours, document the issues, and move away to the next when consistently get no success. Sounds good the plan?

image

@aesteves900 aesteves900 changed the title chore: Upgrade all samples to run with latest uno and latest VS 2022 chore: Upgrade - WebRTC.Wasm Sample to run with latest uno and latest VS 2022 Sep 22, 2022
@jeromelaban
Copy link
Member

Where is this error coming from? Is this for this sample specificall? Can you show the complete error ?

@aesteves900 aesteves900 force-pushed the dev/AEST/152-update-uno-lastest-version branch from 428af62 to 90bc0f9 Compare October 9, 2022 20:38
@aesteves900 aesteves900 marked this pull request as ready for review October 9, 2022 21:00
chore: Upgrade - Web RTC Sample to run with latest uno and latest VS 2022

chore: Upgrade all samples to run with latest uno and latest VS 2022
@aesteves900 aesteves900 force-pushed the dev/AEST/152-update-uno-lastest-version branch from e927e94 to 3a328ca Compare October 12, 2022 18:34
@jeromelaban jeromelaban merged commit 1752a6d into unoplatform:master Oct 13, 2022
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.

None yet

2 participants