Skip to content

README changes for Delete issue#206

Merged
rashmiangadi11 merged 2 commits intodevelopfrom
readmeupdate
Nov 24, 2025
Merged

README changes for Delete issue#206
rashmiangadi11 merged 2 commits intodevelopfrom
readmeupdate

Conversation

@rashmiangadi11
Copy link
Copy Markdown

@rashmiangadi11 rashmiangadi11 commented Nov 23, 2025

Describe your changes

Readme update for SDM user role

Any documentation

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist before requesting a review

  • 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

  • I have Uploaded Screenshots or added lists of the scenarios tested in description

Comment thread README.md Outdated
Here the name of role collection can be choice of yours and role template references can point to application roles.
"$XSSERVICENAME(sdm).SDM_User" should be added to use SDM and "sdm" name should be the instance name from mta.yaml.

Note: If such a role collection is not configured then unauthorized user end up in misusing the attachments.
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.

Can we add this note how we have added other notes : https://github.com/cap-js/sdm/blob/readmeupdate/README.md#steps-to-enable-create-link-feature-in-cap-application like under this section

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

done

@rashmiangadi11 rashmiangadi11 merged commit 2e82963 into develop Nov 24, 2025
9 checks passed
@rashmiangadi11 rashmiangadi11 deleted the readmeupdate branch November 24, 2025 08:09
Juergen-Goebl-msggroup pushed a commit to Andreas-Meindl-msggroup/sdm that referenced this pull request Apr 14, 2026
README changes for Delete issue
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.

4 participants