Skip to content

Conversation

@SarahScargall
Copy link
Contributor

Purpose of the change

The Client Python SDK Introduction page has a Mermaid chart in it, which was not rendering correctly. This PR resolves the issues, so it renders correctly on the docs site.

Description

No Dependencies, single-file change.
Page holding the error:
Docs Client Python SDK Introduction Page

Screenshot of error:
Screenshot 2025-12-04 at 2 26 05 PM

Fixes/Closes

NA

Type of change

[Please delete options that are not relevant.]

  • Bug fix (non-breaking change which fixes an issue)
  • Documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.

  • Manual verification (list step-by-step instructions)
    • run mint dev in lab environment which also runs mintlify and holds a copied branch of the repo.
      Test Results:
      Mint Broken Links passes:
sarah ➜ ~/MemMachine/docs (mermaid) $ mint broken-links
success no broken links found

ScreenShot of correctly rendered Flow Chart in lab environment:
Screenshot 2025-12-04 at 2 28 46 PM

Checklist

[Please delete options that are not relevant.]

  • I have signed the commit(s) within this pull request
  • My code follows the style guidelines of this project (See STYLE_GUIDE.md)
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added unit tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have checked my code and corrected any misspellings

Maintainer Checklist

  • Confirmed all checks passed
  • Contributor has signed the commit(s)
  • Reviewed the code
  • Run, Tested, and Verified the change(s) work as expected

Screenshots/Gifs

[If applicable, add screenshots or GIFs that show the changes in action. This is especially helpful for API responses. Otherwise, delete this section or type "N/A".]

Further comments

[Add any other relevant information here, such as potential side effects, future considerations, or any specific questions for the reviewer. Otherwise, type "None".]

Signed-off-by: sarah <sarah.scargall@memverge.com>
Copy link
Contributor

@sscargal sscargal left a comment

Choose a reason for hiding this comment

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

LGTM

@sscargal sscargal merged commit 128c474 into MemMachine:main Dec 4, 2025
35 checks passed
dxu104 pushed a commit to dxu104/MemMachine that referenced this pull request Dec 5, 2025
Resolving mermaid issue in flow chart

Signed-off-by: sarah <sarah.scargall@memverge.com>
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