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

test using cordova-advanced-http plugin #83

Merged
merged 7 commits into from
Nov 5, 2017
Merged

test using cordova-advanced-http plugin #83

merged 7 commits into from
Nov 5, 2017

Conversation

dman7
Copy link
Member

@dman7 dman7 commented Oct 31, 2017

No description provided.

@dman7
Copy link
Member Author

dman7 commented Oct 31, 2017

@cdparra based on this: https://www.npmjs.com/package/cordova-plugin-advanced-http, are you including the cordovaHTTP service?

With AngularJS
This plugin creates a cordovaHTTP service inside of a cordovaHTTP module. You must load the module when you create your app's module.

var app = angular.module('myApp', ['ngRoute', 'ngAnimate', 'cordovaHTTP']);
You can then inject the cordovaHTTP service into your controllers. The functions can then be used identically to the examples shown below except that instead of accepting success and failure callback functions, each function returns a promise. For more information on promises in AngularJS read the AngularJS docs. For more info on promises in general check out this article on html5rocks. Make sure that you load cordova.js or phonegap.js after AngularJS is loaded.

jorgeramirez and others added 6 commits October 31, 2017 04:07
refs feature/82

Signed-off-by: Jorge Ramírez <jorgeramirez1990@gmail.com>
refs #82

Signed-off-by: Jorge Ramírez <jorgeramirez1990@gmail.com>
refs #82

Signed-off-by: Jorge Ramírez <jorgeramirez1990@gmail.com>
refs #82

Signed-off-by: Jorge Ramírez <jorgeramirez1990@gmail.com>
@cdparra
Copy link
Member

cdparra commented Nov 5, 2017

Issues related to the use of cordovaHTTP seem to be solved now, so I am proceeding to merge this branch

I have, however, identified many other issues (at least one purely backend, ref #84) that should be solved to be fully working. Issues are part of epic #89
cc @dman7 @jorgeramirez

@cdparra cdparra self-assigned this Nov 5, 2017
@cdparra cdparra merged commit 2d9fc5e into master Nov 5, 2017
@cdparra cdparra deleted the feature/82 branch November 5, 2017 22:27
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