SAP / open-ux-tools

Enable community collaboration to jointly promote and facilitate best in class tooling capabilities
Apache License 2.0
83 stars 35 forks source link

FEATURE - support smartlinks configuration in Fiori apps #916

Closed johannes-kolbe closed 1 year ago

johannes-kolbe commented 1 year ago

As a developer, I want to deliver my application with pre-configured targets for the included smart links, so that I can create/modify smart link configurations in my changes folder and deliver together with my app.

Description

This feature request should enable a user to add a smart links configuration to an application. With those pre-configured targets from the system, changes can be done directly and included before deployment.

Technical Design

Acceptance Criteria

GivenWhenThen format (https://martinfowler.com/bliki/GivenWhenThen.html)

Given I have an existing application with system configuration in ui5-deploy.yaml when I execute @sap-ux/create add smartlinks-config command then a smartlinks configuration will be added to the application

Given I have an existing application without ui5-deploy.yaml when I execute @sap-ux/create add smartlinks-config command then I can enter system details and a smartlinks configuration will be added to the application

Given I have a smartlinks configuration in the application when I start flp preview then changes can seen with those targets considered.

Notes

Tasks

tobiasqueck commented 1 year ago

@johannes-kolbe could you add a little more information about what files will be modified or added, maybe a snippet with an example?

johannes-kolbe commented 1 year ago

@tobiasqueck Added more information about the modified files and a snippet of the expected fioriSandboxConfig.json and changes to ui5.yaml

johannes-kolbe commented 1 year ago

@Klaus-Keller For the integration into @sap-ux/create, should a possibility to "remove" the smartlinks configuration be available too? The question arises, if yes, what exactly should be removed/modified then..

Klaus-Keller commented 1 year ago

@johannes-kolbe, for @sap-ux/create it is technically not required to have also a remove option. If it does not really make sense, you can skip it. Perhaps we align with Christoph.

cfg74 commented 1 year ago

@johannes-kolbe @Klaus-Keller I would consider a remove as a prio 2 topic at this point, as an existing configuration doesn't influence something negatively. And update option via "add" is available.