Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@VictorUvarov
Copy link
Contributor

@VictorUvarov VictorUvarov commented Dec 28, 2024

This dependency is not needed

@victortive
Copy link
Contributor

@mikaelwills Mind taking a look?

@mikaelwills
Copy link
Collaborator

@mikaelwills Mind taking a look?

Yep, will have a look on Monday

@VictorUvarov
Copy link
Contributor Author

@mikaelwills Mind taking a look?

Yep, will have a look on Monday

Yeah no worries. I don't actually use this version of the package anymore. My fork has diverged so far because I've removed almost all the dynamic map usage and fixed some internal bugs. Just pushing upstream when I get time.

@mikaelwills
Copy link
Collaborator

@mikaelwills Mind taking a look?

Yep, will have a look on Monday

Yeah no worries. I don't actually use this version of the package anymore. My fork has diverged so far because I've removed almost all the dynamic map usage and fixed some internal bugs. Just pushing upstream when I get time.

Yea the dream would be to have this package completely polished up, I really want to avoid taking on maintaining a fork in my company.

@mikaelwills mikaelwills merged commit 1622e27 into flutter-webrtc:main Jan 4, 2025
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