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

add setDefaultAudioOutput to source #768

Merged
merged 1 commit into from
Apr 17, 2023
Merged

Conversation

jerry2013
Copy link
Contributor

#767 only contains changes to the compiled artifact and not the source - this is to add the same code so that npm run build won't lose the change.

@hthetiot
Copy link
Contributor

@jerry2013 thx for PR.
Please run npm run build and commit changed www file if build and test pass.

@hthetiot
Copy link
Contributor

Good catch.

@hthetiot hthetiot added the bug label Apr 14, 2023
@hthetiot hthetiot added this to the 8.0.x milestone Apr 14, 2023
@jerry2013
Copy link
Contributor Author

did that and the only changes are version number and copyright year.

@hthetiot
Copy link
Contributor

Thx @jerry2013 I will take care of version and copyright. It's not part of PR ;)

@@ -1,9 +1,9 @@
/*
* cordova-plugin-iosrtc v8.0.2
* cordova-plugin-iosrtc v8.0.3
Copy link
Contributor

Choose a reason for hiding this comment

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

This come from package when generated.
It's not the role of a PR to increment version.

Copy link
Contributor Author

@jerry2013 jerry2013 Apr 15, 2023

Choose a reason for hiding this comment

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

Indeed I was very confused why the CHANGELOG.md says 8.0.3 (commit from 10 months ago) but the cordova-plugin-iosrtc.js on the master is 8.0.2.

@hthetiot hthetiot merged commit 7e3bf8a into cordova-rtc:master Apr 17, 2023
@hthetiot
Copy link
Contributor

Released into [email protected] , can you test to confirm your fix please, thx.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants