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

Fixes for Cordova 9 #174

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

dopplerDistortion
Copy link

@dopplerDistortion dopplerDistortion commented Sep 12, 2019

Solves issue(s)

q library added to package.json
context.requireCordovaModule('q') functions changed to require('q')

#165

@allnash
Copy link

allnash commented Oct 3, 2019

Can we please merge this?

I am facing the same issue. People have moved on from cordova 8.

@allnash
Copy link

allnash commented Oct 3, 2019

@Anthonyzou when can we expect to get it merged?

@allnash
Copy link

allnash commented Oct 3, 2019

@jamesdehart who is responsible at OpenTok to approve merge PR's?

@dopplerDistortion
Copy link
Author

Seems like nobody :(

@tusharvikky
Copy link

Any update on when this support will be merged?

kspevak added a commit to trainer-rx/cordova-plugin-opentok that referenced this pull request Nov 27, 2019
@rubenkuipers
Copy link

Can this PR be merged? Would be highly appreciated!

@AAniculaesei-modus
Copy link

@michaeljolley - can anybody look at this ?

@albalooshi
Copy link

@michaeljolley can we get this merged?

Copy link

@assafshp assafshp left a comment

Choose a reason for hiding this comment

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

Can you please merge ?

@michaeljolley michaeljolley self-requested a review July 4, 2020 00:56
@danielraban
Copy link

can we get this merged?

@mheap mheap changed the base branch from master to main March 22, 2021 09:50
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.