-
Notifications
You must be signed in to change notification settings - Fork 72
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
Serve application projects listed as project dependencies in /resources #45
Comments
Instead of enhancing the application type, @matz3 and I discussed the possibility of adding "component" types which are always served at their namespace (e.g. Application projects would then only act as an entry point or test environment (e.g. the OpenUI5 testsuite, which is actually not a UI5 application) and will continue to be served at This setup would be closer to how many apps are being developed, especially in the Fiori Launchpad context. |
hi Currently we are doing a workaround with defining the Project as shim
this comes with the problem that the shims in the sub project are not applyed corrertly they are applyed but on root level an not the level of the subproject so following shim inside the
will result in the following scructur
|
@sebbi08 The virtual path defined for a To get the expected behavior, your shim would need to define the virtual path as Namespaces, "root paths" or other hierarchy-like attributes are not shared between a project and its dependencies (even if they are shimmed). |
Hi @RandomByte , do you plan to add the "component" types in the near future? #45 (comment) Do we have any other solution for this issue? Thanks |
Hey @resitamas, this is one of many topics we are currently pursuing. So I can't give you an ETA. However we see great demand in such a type. Currently we are in discussion on how such a "component" project should be structured. I.e. whether it should look like the current "application" projects (one "webapp" directory, containing a "test" directory) or more like the current "library" projects with separate "src" and "test" directories and a deep directory hierarchy representing the projects namespace. We tend towards something in between, like a flat hierarchy (without the namespace) with separate "src" and "test" directories. |
Hi @RandomByte, this would be very helpful for a local "mock launchpad", hosting multiple applications and testing cross navigation locally. From: https://sap.github.io/ui5-tooling/pages/Builder/#types Projects of type application are typically the main or root project. In a projects dependency tree, there should only be one project of type application. If multiple are found, those further away from the root are ignored. I'm wondering if it would be possible for those further away from the root to be served under their namespace. |
Hi @RandomByte, Hi @matz3, will this also support a monorepo project structure? E.g. I have a monorepo with multiple apps, then I can execute |
Hi @RandomByte, is it considered to add the |
It is unfortunately out of scope for the initial 4.0 release, but as long as it doesn't require breaking changes it might come in a later 4.x minor version. |
Expected Behavior
In some use cases, it is desired to use multiple ui5 components in a shared environment (for instance loading an application component appB file during runtime once a tile was clicked in appA). This is currently possible with HTML5 / ui5 applications that are deployed in a SCP account, in conjunction with a configuration in
neon-app.json
.Current Behavior
As of now, ui5 application projects won’t be served in
/resources
when runningui5 serve
in a ui5 root project, even when they contain anui5.yaml
descriptor file and are listed as dependencies.Steps to reproduce the issue
Suppose the following structure of two ui5 application projects:
A global link of the
openui5-sample
app is created:The sample app is added as a dependency
The ui5-sample-app is made known in the root project:
However, when running
ui5 serve
in my-root-app, requests that point to the openui5-sample-app will fail.The text was updated successfully, but these errors were encountered: