Skip to content

Update TMS documentation - #2907

Merged
ecklie merged 6 commits into
mainfrom
mtx-tms-migration-hint
Sep 23, 2026
Merged

ecklie merged 6 commits into
mainfrom
mtx-tms-migration-hint

Conversation

@ecklie

@ecklie ecklie commented Sep 18, 2026

Copy link
Copy Markdown
Contributor
  • remove migration hint
  • add link to hana migration documentation
  • add recommendation for a switch to tms without redeployment

@hyperspace-pr-bot

Copy link
Copy Markdown
Contributor

👋 Hi — I'm PR Bot, your SAP code review assistant.

I'll automatically review your pull requests for code quality, security, and SAP compliance. Get an overview of what I do →

What I do

  • Summarize your pull request changes
  • Review code for quality, correctness, and reliability
  • Suggest fixes when a pipeline job fails

Key commands

Command Description
/review [--all] Trigger a code review. Add --all to include files excluded by excluded_paths.
/summarize Generate a PR summary
/ask <question> Ask about the current changes
/help See all available commands

*This introduction message will be shown to you only once, you will not see it in future PRs.

@hyperspace-pr-bot

Copy link
Copy Markdown
Contributor

Control Panel

Hi, I'm an AI-powered Review Bot that helps you with summarizing and reviewing pull requests.
To interact with me, just use the following actions:

  • 📝 Summarize PR
  • 🔍 Review
  • 🗑️ Delete all bot comments and reviews

@ecklie
ecklie marked this pull request as ready for review September 18, 2026 12:24

@hyperspace-pr-bot hyperspace-pr-bot Bot left a comment

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.

I posted five inline documentation review comments on the changed section, focused on typos, grammar, clarity, durable wording, and more precise terminology. Overall, the update is helpful, but the new migration section needs a few small edits before merge so it is easier to scan and less likely to become stale.

PR Bot Information

Version: 1.31.43

Comment thread guides/multitenancy/index.md Outdated
Comment thread guides/multitenancy/index.md Outdated
Comment thread guides/multitenancy/index.md Outdated
Comment thread guides/multitenancy/index.md Outdated
Comment thread guides/multitenancy/index.md Outdated
@hyperspace-pr-bot hyperspace-pr-bot Bot deleted a comment from ecklie Sep 18, 2026
ecklie and others added 2 commits September 18, 2026 14:31
Co-authored-by: hyperspace-pr-bot[bot] <209611008+hyperspace-pr-bot[bot]@users.noreply.github.com>
@chgeo chgeo changed the title update MTX TMS documentation Update TMS documentation Sep 21, 2026

@hyperspace-pr-bot hyperspace-pr-bot Bot left a comment

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.

I found one remaining typo in the new note and left an inline suggestion. The earlier wording and clarity comments appear to have been addressed; please also apply the remaining prior suggestion about “service instance” if it has not been updated yet.

PR Bot Information

Version: 1.31.43

Comment thread guides/multitenancy/index.md Outdated
@hyperspace-pr-bot hyperspace-pr-bot Bot deleted a comment from ecklie Sep 21, 2026
ecklie and others added 2 commits September 21, 2026 17:38
Co-authored-by: hyperspace-pr-bot[bot] <209611008+hyperspace-pr-bot[bot]@users.noreply.github.com>
@ecklie
ecklie requested a review from swaldmann September 23, 2026 08:45
@ecklie
ecklie added this pull request to the merge queue Sep 23, 2026
Merged via the queue into main with commit d0ea503 Sep 23, 2026
4 checks passed
@ecklie
ecklie deleted the mtx-tms-migration-hint branch September 23, 2026 09:21
@ecklie

ecklie commented Sep 23, 2026

Copy link
Copy Markdown
Contributor Author

@smahati Can we add this to the next patch? Is there a way to mark PRs for the next patch?

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.

2 participants