Skip to content
This repository has been archived by the owner on Jan 18, 2023. It is now read-only.

v3.3.0 #97

Closed
wants to merge 13 commits into from
Closed

v3.3.0 #97

wants to merge 13 commits into from

Conversation

msach22
Copy link

@msach22 msach22 commented May 7, 2018

Contributing checklist

  • Code must follow existing styling conventions
  • Added a descriptive commit message

Solves issue(s)

@mark-veenstra
Copy link

@msach22 Can you also take #90 and #69 in this release?

@mark-veenstra
Copy link

@msach22 Wouldn't it be more clear to pin the JS SDK version to the one that will be installed? Maybe it is clearer to bump the version in package.json also and remove the ^: https://github.com/opentok/cordova-plugin-opentok/blob/3.3.0/package.json#L14

@msach22
Copy link
Author

msach22 commented May 7, 2018

@mark-veenstra I will review #90 and #69!

As for the JS SDK, the 2.14.0 release had some issues so we rolled back. We plan on releasing our 2.14.1 patch this week so I will add that to this release as well!

@msach22 msach22 closed this May 17, 2018
@msach22 msach22 deleted the 3.3.0 branch May 17, 2018 18:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants