-
Notifications
You must be signed in to change notification settings - Fork 71
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
New builder setting to fix data source URI in manifest.json #625
Comments
It would be great to have this feature directly in ui5-tooling. At the moment the Fiori Application Generator uses a custom task using "webide-extension-task-updateManifestJson" to update the manifest.json when deploying to Cloud Foundry. https://www.npmjs.com/package/@sap/ui5-builder-webide-extension is quite old and has a dependency to @ui5/[email protected] and @ui5/[email protected]. |
Absolutely agree with previous post, current 'deploy-config' script generates quite an outdated and huge list of dependencies. |
As far as I remember, SAP Build Work Zone, standard edition a.k.a "Central Launchpad on CF" does not support absolute URLs at all. So the url should be relative in the source manifest.json. I wonder why you not adjust the source manifest.json to make it relative right from the beginning? |
Hi everyone, |
Hi @nicoschoenteich, So as long as we have no auto correction available, feel free to go with the custom solution :-) |
Hi @flovogt, I implemented it as part of the |
Is your feature request related to a problem? Please describe.
Data source URI in manifest.json should not start with a slash in order to UI5 application work properly in a FLP.
Describe the solution you'd like
It would be nice to have a builder task or setting that checks manifest.json and strips off leading slash from a data source URI during a building of an application to ensure that URI is correct automatically.
Describe alternatives you've considered
There is a such feature in @sap/grunt-sapui5-bestpractice-build module that is not supported any more and should be replaced with @ui5/builder.
Additional context
Example of a data source section of manifest.json
The text was updated successfully, but these errors were encountered: