Skip to content

Conversation

@rishikunnath2747
Copy link
Collaborator

@rishikunnath2747 rishikunnath2747 commented Dec 18, 2025

This PR adds support for consumer applications to translate the error messages propagated by the SDM plugin
The error messages are fetched based on the application language settings and cached when ReadAttachmentsHandler runs
Subsequently it fetches all the translated error messages from the cache
In case the consumer application doesn't define translated error messages, it defaults to the error messages provided in the SDMErrorMessages class in english

Type of change

Please delete options that are not relevant.

  • This change requires a documentation update

Checklist before requesting a review

  • I follow Java Development Guidelines for SAP
  • I have tested the functionality on my cloud environment.
  • I have provided sufficient automated/ unit tests for the code.
  • I have increased or maintained the test coverage.
  • I have ran integration tests on my cloud environment.
  • I have validated blackduck portal for any vulnerability after my commit.

Upload Screenshots/lists of the scenarios tested

Single tenant : https://github.com/cap-java/sdm/actions/runs/20517677888/job/58947951889
Multi tenant : https://github.com/cap-java/sdm/actions/runs/20517682015/job/58949765104

@rishikunnath2747 rishikunnath2747 added the Under Construction The PR is still under construction label Dec 18, 2025
@yashmeet29 yashmeet29 marked this pull request as ready for review December 22, 2025 08:07
@rishikunnath2747 rishikunnath2747 removed the Under Construction The PR is still under construction label Dec 26, 2025
yashmeet29
yashmeet29 previously approved these changes Dec 26, 2025
@rishikunnath2747 rishikunnath2747 merged commit 0fc7bea into develop Dec 26, 2025
9 checks passed
@rishikunnath2747 rishikunnath2747 deleted the SDMEXT-2265 branch December 26, 2025 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants