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

[TIMOB-6193] Changed Ti.include() to use GET instead of POST. #704

Merged
merged 1 commit into from Nov 16, 2011
Merged

[TIMOB-6193] Changed Ti.include() to use GET instead of POST. #704

merged 1 commit into from Nov 16, 2011

Conversation

cb1kenobi
Copy link
Contributor

No description provided.

@donthorp
Copy link
Contributor

Code reviewed. Request Accepted

@nebrius
Copy link
Contributor

nebrius commented Nov 16, 2011

Testing using the built in web-server, and it doesn't work (as mentioned in the Jira ticket). Is this because of timob-6194 (relative paths)?

I tested using my own node.js server (using node-static file server module) with the document root pointed at /build/mobileweb such that the URL in the browser was http://localhost:8080/index.html and it worked fine.

Once I get confirmation on this issue, I will be ready to move forward on this pull request.

@nebrius
Copy link
Contributor

nebrius commented Nov 16, 2011

Tested. Request Accepted

donthorp added a commit that referenced this pull request Nov 16, 2011
[TIMOB-6193] Changed Ti.include() to use GET instead of POST.
@donthorp donthorp merged commit 1ab5afc into tidev:master Nov 16, 2011
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.

None yet

3 participants