openui5 icon indicating copy to clipboard operation
openui5 copied to clipboard

Tutorial Bug: Downloaded demo project doesn't work OOB

Open HymanZHAN opened this issue 2 years ago • 1 comments

Steps to reproduce the problem:

  1. Download the demo app from https://sdk.openui5.org/entity/sap.m.tutorial.walkthrough/sample/sap.m.tutorial.walkthrough.30
  2. Unzip the project, go to the project root and run npm i
  3. Try to start the project with npm start

What is the expected result?

The project should start successfully.

What happens instead?

ui5 serve failed miserably.

Any other information?

> npm start

> [email protected] start
> ui5 serve


⚠️  Process Failed With Error

Error Message:
Could not find required manifest.json for project sap.m.tutorial.walkthrough.30: ENOENT: no such file or directory, open 'C:\Users\I572233\Downloads\sap.m.tutorial.walkthrough.30\manifest.json'

For details, execute the same command again with an additional '--verbose' parameter

Apparently, the location of manifest.json is incompatible with the config of ui5.yaml, where it says:

specVersion: '2.3'
metadata:
  name: "sap.m.tutorial.walkthrough.30"
type: application
resources:
  configuration:
    paths:
      webapp: .

webapp is configured to the project root, while manifest.json still lives inside the webapp folder.

Additional issues

After starting the downloaded demo project, the following error occurs.

image

Apparently, we don't have a resource folder like the online live project.

image

These issues plague almost every downloaded demo project in the tutorial and can confuse someone new to opnui5 and SAPUI5. Please fix the tutorial's downloadable demo code. Thanks!

HymanZHAN avatar Aug 12 '22 01:08 HymanZHAN

Hello @HymanZHAN, Thank you for sharing this finding. I've created an internal incident 2280160353. The status of the issue will be updated here in GitHub. Regards, Konstantin

kskondov avatar Aug 12 '22 13:08 kskondov

Issue is not reproducible anymore as probably one of the changes on the involved code since it was opened has fixed it.

jdichev avatar Nov 08 '22 08:11 jdichev