Skip to content

fix: Translation texts for Add Data Source and SAPUI5 Model title and subtitle#4573

Open
mmilko01 wants to merge 10 commits intomainfrom
fix/add-new-model-labels
Open

fix: Translation texts for Add Data Source and SAPUI5 Model title and subtitle#4573
mmilko01 wants to merge 10 commits intomainfrom
fix/add-new-model-labels

Conversation

@mmilko01
Copy link
Copy Markdown
Contributor

@mmilko01 mmilko01 commented Apr 17, 2026

#4577

  • Fixed spelling error in title and subtitle
  • Enhanced validation error message for Data Source and SAPUI5 model name prompt

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 17, 2026

🦋 Changeset detected

Latest commit: 68afc9c

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

This PR includes changesets to release 11 packages
Name Type
@sap-ux/adp-tooling Patch
@sap-ux/generator-adp Patch
@sap-ux/abap-deploy-config-sub-generator Patch
@sap-ux/adp-flp-config-sub-generator Patch
@sap-ux/backend-proxy-middleware-cf Patch
@sap-ux/create Patch
@sap-ux/flp-config-inquirer Patch
@sap-ux/preview-middleware Patch
@sap-ux/deploy-config-sub-generator Patch
@sap-ux/flp-config-sub-generator Patch
@sap-ux-private/preview-middleware-client 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

@mmilko01 mmilko01 changed the title fix: translation texts fix: Adapt Add OData Service and SAPUI5 Model name after CF support Apr 20, 2026
@mmilko01 mmilko01 changed the title fix: Adapt Add OData Service and SAPUI5 Model name after CF support fix: Translation texts for Add Data Source and SAPUI5 Model title and subtitle Apr 20, 2026
@mmilko01 mmilko01 marked this pull request as ready for review April 20, 2026 15:07
@mmilko01 mmilko01 requested review from a team as code owners April 20, 2026 15:07
@hyperspace-insights
Copy link
Copy Markdown
Contributor

Summary

The following content is AI-generated and provides a summary of the pull request:


Fix: Translation Texts for "Add Data Source and SAPUI5 Model" Title, Subtitle, and Validation Message

Bug Fix

🐛 Corrected a spelling error in the title and subtitle of the "Add Data Source and SAPUI5 Model" generator (was incorrectly spelled as "Add Datasource and SAPUI5 Model"). Also updated the validation error message for duplicate data source and model names to be more accurate and relevant.

Changes

  • .changeset/cyan-glasses-own.md: Added changeset entry for the patch fix in @sap-ux/adp-tooling and @sap-ux/generator-adp.
  • packages/adp-tooling/src/translations/adp-tooling.i18n.json: Updated errorDuplicatedValueOData validation message from "An OData annotation or service with the same name was already added to the project. Rename and try again." to "Data Source and SAPUI5 model with the same name were already added to the project. Rename and try again.".
  • packages/adp-tooling/test/unit/prompts/add-new-model/index.test.ts: Updated the expected validation error message in the test to match the corrected translation text.
  • packages/generator-adp/src/translations/generator-adp.i18n.json: Fixed spelling of addNewModelName and addNewModelDescr from "Add Datasource..." to "Add Data Source...".
  • packages/generator-adp/src/types.ts: Updated the ADD_NEW_MODEL enum value from 'Add Datasource and SAPUI5 Model' to 'Add Data Source and SAPUI5 Model'.

GitHub Issues

  • #4577: BUG - Fix translation texts for Add Data Source and SAPUI5 Model generator title and subtitle

  • 🔄 Regenerate and Update Summary
  • ✏️ Insert as PR Description (deletes this comment)
  • 🗑️ Delete comment
PR Bot Information

Version: 1.20.11 | 📖 Documentation | 🚨 Create Incident | 💬 Feedback


💌 Have ideas or want to contribute? Create an issue and share your thoughts with us!
📑 Check out the documentation for more information.
📬 Subscribe to the Hyperspace PR Bot DL to get the latest announcements and pilot features!

Made with ❤️ by Hyperspace.

Copy link
Copy Markdown
Contributor

@hyperspace-insights hyperspace-insights Bot left a comment

Choose a reason for hiding this comment

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

The PR is a clean, focused text correction — fixing "Datasource" → "Data Source" in display strings, updating the errorDuplicatedValueOData validation message to be more descriptive, and keeping the test in sync. The changes are consistent across the i18n files, enum, and test. The only pre-existing issue noted is the "Cloud Foundary" typo in the deploy section, but that is unrelated to the lines changed in this PR and was flagged incorrectly on line 73 (the wrong location). Overall the PR is well-scoped and the changes are correct.

PR Bot Information

Version: 1.20.11 | 📖 Documentation | 🚨 Create Incident | 💬 Feedback

  • File Content Strategy: Full file content
  • Agent Instructions:
  • Correlation ID: ca3216b8-09b5-4d0e-bc35-cb4418577230
  • Event Trigger: pull_request.ready_for_review
  • LLM: anthropic--claude-4.6-sonnet

Comment thread packages/adp-tooling/src/translations/adp-tooling.i18n.json Outdated
nikmace
nikmace previously approved these changes Apr 21, 2026
Copy link
Copy Markdown
Contributor

@nikmace nikmace left a comment

Choose a reason for hiding this comment

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

Changes look good
Coverage is good
Changeset is OK
Did not test manually

"fileDoesNotExist": "This file does not exist.",
"annotationFileAlreadyExists": "There is already an annotation file with the same name. Please choose another file or rename the file and select it again.",
"errorDuplicatedValueOData": "An OData annotation or service with the same name was already added to the project. Rename and try again.",
"errorDuplicatedValueOData": "Data Source and SAPUI5 model with the same name were already added to the project. Rename and try again.",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
"errorDuplicatedValueOData": "Data Source and SAPUI5 model with the same name were already added to the project. Rename and try again.",
"errorDuplicatedValueOData": "A data source and SAPUI5 model with the same name were already added to the project. Rename and try again.",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed in: 6922d07

"addNewModelName": "Add Datasource and SAPUI5 Model",
"addNewModelDescr": "Select a Datasource and SAPUI5 model.",
"addNewModelName": "Add Data Source and SAPUI5 Model",
"addNewModelDescr": "Select a Data Source and SAPUI5 model.",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
"addNewModelDescr": "Select a Data Source and SAPUI5 model.",
"addNewModelDescr": "Select a data source and SAPUI5 model.",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed in: 682e87d

Co-authored-by: Louise Findlay <[email protected]>
@nikmace nikmace requested review from lfindlaysap and nikmace April 21, 2026 13:21
Copy link
Copy Markdown
Contributor

@nikmace nikmace left a comment

Choose a reason for hiding this comment

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

Re-approve ✅

@sonarqubecloud
Copy link
Copy Markdown

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

Labels

adp-tooling generator-adp @sap-ux/generator-adp

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants