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

wdi5 ts init adds wrong ui5 type devDependency #610

Closed
heimwege opened this issue May 7, 2024 · 2 comments
Closed

wdi5 ts init adds wrong ui5 type devDependency #610

heimwege opened this issue May 7, 2024 · 2 comments

Comments

@heimwege
Copy link
Contributor

heimwege commented May 7, 2024

Describe the bug
The easy ui5 generator includes a sub-generator that handles the wdi5 integration into the generated app. This mostly consists of npm init wdi5 -- --ts. Init wdi5 seems to hard coded introduce a devDependency to the latest version of @openui5/types. But in the context of the easy ui5 generator the user can select between sapui5 and openui5 and also which specific version to be used.

See also my initial issue for the ui5 ts app generator: ui5-community/generator-ui5-ts-app#33

I think this is caused here: https://github.com/ui5-community/create-wdi5/blob/main/src%2Findex.ts#L20 so maybe this is the wrong repo for this issue 🙈

To Reproduce
yo easy-ui5 -> select generator ts-app and follow the instructions. select sapui5 (e.g. 1.120.11) and have a look at the package.json afterwards.

Expected behavior
Depending on the selection in the easy ui5 generator either @openui5/types or @sapui5/types should be included as devDependency in the version entered in the respective generator step.

@petermuessig
Copy link
Member

Hi @heimwege

Yes, this is the wrong repo - it should be moved to the create-wdi5 repository. But the issue is valid. In case of the UI5 generators added the types already, they must not be added additionally. This should be checked in the generator steps.

@heimwege
Copy link
Contributor Author

heimwege commented May 8, 2024

Thanks for the confirmation @petermuessig I created a new issue in the wdi5-create repo. Let's close this one.

@heimwege heimwege closed this as completed May 8, 2024
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

No branches or pull requests

2 participants