-
Notifications
You must be signed in to change notification settings - Fork 506
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
Compose with cds-plugins #555
base: main
Are you sure you want to change the base?
Conversation
app.serve ('/data') .from('@capire/data-viewer','app/viewer') | ||
app.serve ('/admin-authors') .from ('@capire/fiori','app/admin-authors') | ||
app.serve ('/admin-books') .from ('@capire/fiori','app/admin-books') | ||
app.serve ('/browse-books') .from ('@capire/fiori','app/browse') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these apps are not loading correctly by starting cds w
from compose or bookstore yet
@capire/fiori annotations for UIs are only loaded when starting with cds w fiori
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apps are not loaded correctly, as they don't have an index.html file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in ./fiori they are loaded through a fiori-apps.html
However using this (e.g. app.serve ('/fiori-apps') .from ('@capire/fiori','app/fiori-apps.html')
)instead doesn't work either, because the fioriSandboxConfig.json
is not found, hence it's showing a generic FLP with random apps missing the ones we would want to show.
Therefore, we need to copy the app/appconfig/fioriSandboxConfig.json
to achieve a similar behavior
do we need to copy from |
…ples/cloud-cap-samples into compose-with-cds-plugins
I think now |
No description provided.