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

Eliminate circular dependency to fix building with Vite #672

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

spaun
Copy link

@spaun spaun commented Nov 22, 2021

No description provided.

@CSantosM CSantosM force-pushed the master branch 2 times, most recently from 7fe1145 to d3a755b Compare May 12, 2022 08:27
@joker-x
Copy link

joker-x commented May 23, 2022

I am very interesting in this pull request was merged if it works fine. Can I help?

@manoelcfeliciano
Copy link

manoelcfeliciano commented Aug 2, 2022

I can certify that the changes suggested by @spaun works as expected. I ported his changes to the latest version of OpenVidu (2.22.0) and got it working on my projects.

I guess we can merge this without any problem. Any thoughts from the maintainers here would be appreciated.

@pabloFuente @CSantosM @j1elo @cruizba

@CSantosM CSantosM force-pushed the master branch 10 times, most recently from e9de7f5 to 7d88c1f Compare October 5, 2022 09:54
@CSantosM CSantosM force-pushed the master branch 2 times, most recently from 670697a to 6ab6c22 Compare November 16, 2022 11:15
@CSantosM CSantosM force-pushed the master branch 3 times, most recently from bc3613c to 86d1260 Compare November 25, 2022 11:25
@CSantosM CSantosM force-pushed the master branch 2 times, most recently from 4df7846 to 5349177 Compare December 20, 2022 10:47
@CSantosM CSantosM force-pushed the master branch 3 times, most recently from 5847916 to 0cff58e Compare June 21, 2023 13:39
@CSantosM CSantosM force-pushed the master branch 10 times, most recently from bfa4d17 to 33d1192 Compare July 30, 2024 16:13
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.

3 participants