Skip to content

Conversation

johannes-kolbe
Copy link
Contributor

Adds telemetry instructions to the MCP server

Copy link

changeset-bot bot commented Oct 14, 2025

🦋 Changeset detected

Latest commit: 9360523

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

This PR includes changesets to release 1 package
Name Type
@sap-ux/fiori-mcp-server 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

@johannes-kolbe johannes-kolbe marked this pull request as ready for review October 15, 2025 11:15
heimwege
heimwege previously approved these changes Oct 17, 2025
Copy link
Contributor

@heimwege heimwege left a comment

Choose a reason for hiding this comment

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

Readme adjustment looks good. Thanks @johannes-kolbe
Do we need a changeset? Or rather wait for the next change to publish this to npmjs?

devinea
devinea previously approved these changes Oct 20, 2025
Copy link
Member

@devinea devinea left a comment

Choose a reason for hiding this comment

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

changes lg readme.md ✅
no changeset is that intentional?

@johannes-kolbe johannes-kolbe dismissed stale reviews from devinea and heimwege via 05cdffe October 20, 2025 12:22
@johannes-kolbe
Copy link
Contributor Author

johannes-kolbe commented Oct 20, 2025

@devinea @heimwege thanks, changeset added - could you please re-review?

Copy link
Member

@devinea devinea left a comment

Choose a reason for hiding this comment

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

Re-approving with changeset

Copy link
Contributor

@heimwege heimwege 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

Copy link

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.

3 participants