openui5 icon indicating copy to clipboard operation
openui5 copied to clipboard

Regarding the OpenUI5 CDN hosts

Open pubmikeb opened this issue 2 years ago • 6 comments

Which host should be used for the UI5 CDN:

  • https://sdk.openui5.org/resources/sap-ui-core.js
  • https://openui5.hana.ondemand.com/resources/sap-ui-core.js

Does the introduction of https://sdk.openui5.org mean that https://openui5.hana.ondemand.com is planned to be shutted down?

pubmikeb avatar Sep 06 '22 19:09 pubmikeb

This blog post by Oliver Graeff might help: Convenient URLs for SAPUI5/OpenUI5 CDN

boghyon avatar Sep 06 '22 21:09 boghyon

The blog clarifies the subject.

pubmikeb avatar Sep 07 '22 06:09 pubmikeb

Possible bug / behavioral inconsistency in the new CDN URLs.

There are two NIGHTLY-build pages:

  • https://openui5nightly.hana.ondemand.com/topic/99ac68a5b1c3416ab5c84c99fefa250d
  • https://sdk.openui5.org/nightly/#/topic/99ac68a5b1c3416ab5c84c99fefa250d

Both of these pages have a button with a title «What's New in Version 1.106». When I click on this button from https://openui5nightly.hana.ondemand.com/topic/99ac68a5b1c3416ab5c84c99fefa250d, then I'm forwarded to https://openui5nightly.hana.ondemand.com/topic/c70bb907d05a4f9fb6c36ecf73b3fb2e.html and that page really has the wanted content (What's New in OpenUI5 1.106), it's OK.

But when I click on the same button on https://sdk.openui5.org/nightly/#/topic/99ac68a5b1c3416ab5c84c99fefa250d, then I'm forwarded to https://sdk.openui5.org/nightly/topic/c70bb907d05a4f9fb6c36ecf73b3fb2e.html and there I get an error: «The requested resource was not found.» The most important, that if I click on the Documentation menu item (left-upper corner) on this page, then I'm forwarded to the NON-NIGHTLY build version — https://sdk.openui5.org/topic.

pubmikeb avatar Sep 07 '22 15:09 pubmikeb

Hi @pubmikeb , good catch! I've created an internal incident 2270118936. The status of the issue will be updated here in GitHub. Thank you for sharing this finding

flovogt avatar Sep 08 '22 06:09 flovogt

Hi @pubmikeb , the problem seems to be fixed now, can you plesae re-check and confirm? Thank you!

Best regards, Petya Markova. (UI5 Developer)

PetyaMarkovaBogdanova avatar Oct 07 '22 12:10 PetyaMarkovaBogdanova

@PetyaMarkovaBogdanova, still not fixed.

Repro:

  1. Open https://openui5nightly.hana.ondemand.com/topic/99ac68a5b1c3416ab5c84c99fefa250d
  2. Click on «What's New in OpenUI5 1.107»
  3. https://openui5nightly.hana.ondemand.com/topic/b88b40e124634bb4897f36846f23cd12.html will be opened
  4. I see the content of «What's New in OpenUI5 1.107», that's OK
  5. Open https://sdk.openui5.org/nightly/#/topic/99ac68a5b1c3416ab5c84c99fefa250d
  6. Click on «What's New in OpenUI5 1.107»
  7. https://sdk.openui5.org/nightly/topic/b88b40e124634bb4897f36846f23cd12.html will be opened
  8. I see «Not found» / «The requested resource was not found.», that's a bug:

08_133317

BTW, for some reason, after opening https://sdk.openui5.org/nightly/topic/b88b40e124634bb4897f36846f23cd12.html the version number pn the top right corner has changed from «Version 1.108.0 - development in progress.» to «Version 1.107.1 (Latest)».

P.S. I faced the same issue on some other pages on https://sdk.openui5.org/nightly, unfortunately, don't remember where exactly (API Ref or Samples), but still this issue should be rechecked, perhaps, there are more broken links.

pubmikeb avatar Oct 08 '22 11:10 pubmikeb

Hi @pubmikeb , since we have other navigation issues addressed for Demo Kit, when application is contained in a subdirectory, we created a backlog item BGSOFUIPIRIN-5524 to dig in the problem and solve it centrally.

Thanks for the finding! Best regards, Petya Markova. (UI5 Developer)

PetyaMarkovaBogdanova avatar Oct 28 '22 10:10 PetyaMarkovaBogdanova

Let's keep this issue open until the actual issue is fixed.

flovogt avatar Oct 28 '22 11:10 flovogt

I'm not able to reproduce the issue anymore. Probably fixed by https://github.com/SAP/openui5/commit/bbc633b5e1b0917ce4a578d0fadcf7fb1e903aa8.

boghyon avatar Jan 11 '24 13:01 boghyon