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

The "TDG" (the definite guide) demo app fails #52

Closed
akudev opened this issue May 13, 2014 · 4 comments
Closed

The "TDG" (the definite guide) demo app fails #52

akudev opened this issue May 13, 2014 · 4 comments

Comments

@akudev
Copy link
Contributor

akudev commented May 13, 2014

The "TDG" (the definite guide) demo app (https://openui5.hana.ondemand.com/test-resources/sap/m/demokit/tdg/index
.html) demonstrating the newly documented "Application Best Practices" fails to load because the proxy in uilib-sample is hardcoded, so it accesses
https://openui5.hana.ondemand.com/uilib-sample/proxy/http/services.odata
.org/V2/(S(sapuidemotdg))/OData/OData.svc/$metadata

(UI5 version 1.20.5)

Does odata.org support CORS? Then it might be sufficient to check for
the existence of the proxy with HEAD request and just not use the proxy
when it is not there.

@akudev
Copy link
Contributor Author

akudev commented May 13, 2014

Internally tracked as 0120031469 0000491357 2014

@akudev
Copy link
Contributor Author

akudev commented May 28, 2014

Hi all,

as a temporary workaround there is a fixed version of the TDG app available here now:
https://openui5test.hana.ondemand.com/test-resources/sap/m/demokit/tdg/index.html?responderOn=true

Regards
Andreas

@akudev
Copy link
Contributor Author

akudev commented Jun 16, 2014

The app now works fine at the original location.
NOTE: a URL parameter is required to make sure it uses a mock service to circumvent cross-domain issues:
https://openui5.hana.ondemand.com/test-resources/sap/m/demokit/tdg/index.html?responderOn=true

Keeping open until all links and the non-mock version are proper.

@akudev
Copy link
Contributor Author

akudev commented Aug 7, 2014

@akudev akudev closed this as completed Aug 7, 2014
@akudev akudev added fixed and removed approved labels Aug 7, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant