Skip to content

Conversation

nikmace
Copy link
Contributor

@nikmace nikmace commented Oct 6, 2025

Feat for #3699.

  • Removes .adp/config.json.
  • Adds fiori-preview and fiori-proxy middleware.
  • Adds start script for starting the preview in a new tab.

@nikmace nikmace self-assigned this Oct 6, 2025
@nikmace nikmace requested a review from a team as a code owner October 6, 2025 14:02
@nikmace nikmace added feature New feature or request adp-tooling generator-adp @sap-ux/generator-adp labels Oct 6, 2025
Copy link

changeset-bot bot commented Oct 6, 2025

🦋 Changeset detected

Latest commit: d06b55b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 7 packages
Name Type
@sap-ux/generator-adp Patch
@sap-ux/adp-tooling Patch
@sap-ux/adp-flp-config-sub-generator Patch
@sap-ux/create Patch
@sap-ux/flp-config-inquirer Patch
@sap-ux/preview-middleware Patch
@sap-ux/flp-config-sub-generator Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Comment on lines 2 to 3
'@sap-ux/project-input-validator': patch
'@sap-ux/generator-adp': patch
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Is it needed for the @sap-ux/project-input-validator to be added in the changeset? I couldn't see any changes to the package?
  • Similar for @sap-ux/generator-adp, may be there is no need for patch to be released for a snapshot update only?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, this is what was recommended by cset. I'll remove @sap-ux/project-input-validator. The @sap-ux/generator-adp is needed because it consumes the @sap-ux/adp-tooling package, and it will be included anyway in here:
Screenshot 2025-10-08 at 15 50 10

@nikmace nikmace requested a review from mmilko01 October 9, 2025 06:09
Base automatically changed from feat/3514/cf-business-logic to main October 13, 2025 14:29
@nikmace nikmace requested a review from a team as a code owner October 13, 2025 14:29
Copy link
Contributor

@lfindlaysap lfindlaysap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nikmace, one suggestion.

@nikmace nikmace requested a review from lfindlaysap October 14, 2025 06:04
lfindlaysap
lfindlaysap previously approved these changes Oct 14, 2025
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adp-tooling feature New feature or request generator-adp @sap-ux/generator-adp pr:do-not-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants