Skip to content

feat: implement dynamic routing for Sistent components#7408

Open
rishiraj38 wants to merge 2 commits intolayer5io:masterfrom
rishiraj38:feat/sistent-refactor-mdx
Open

feat: implement dynamic routing for Sistent components#7408
rishiraj38 wants to merge 2 commits intolayer5io:masterfrom
rishiraj38:feat/sistent-refactor-mdx

Conversation

@rishiraj38
Copy link
Contributor

Description

This PR refactors the Sistent component documentation system to migrate from a hardcoded data source (content.js) to a dynamic, MDX-driven approach using Gatsby's data layer. This change improves scalability and maintainability of the design system documentation.

Signed commits

  • Yes, I signed my commits.

@github-actions github-actions bot added area/projects An issue relating to Layer5 initiatives (projects) area/site-config project/sistent labels Feb 5, 2026
Migrate Sistent components to MDX-based dynamic routing. This removes the hardcoded content.js and replaces it with a scalable, GraphQL-driven approach suitable for large component libraries.

Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@rishiraj38 rishiraj38 force-pushed the feat/sistent-refactor-mdx branch from 860abe1 to b7bc06b Compare February 5, 2026 13:12
@rishiraj38
Copy link
Contributor Author

@Rajesh-Nagarajan-11 Could you please review and give some feedback?

@l5io
Copy link
Contributor

l5io commented Feb 5, 2026

🚀 Preview for commit b7bc06b at: https://69849a8800c6f13b7bbb8e77--layer5.netlify.app

Migrated all Sistent components to dynamic MDX routing. Standardized component header layouts by moving descriptions to frontmatter. Cleaned up legacy content.js and deprecated code.

Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@rishiraj38
Copy link
Contributor Author

@Rajesh-Nagarajan-11 Could you please review this? Is this the correct way?

@rishiraj38
Copy link
Contributor Author

This needs a bit more focus. I just need a review to see if I’m moving in the right direction or if I’m increasing your workload. 😅

@Rajesh-Nagarajan-11
Copy link
Member

present it in monday's website meeting please

@l5io
Copy link
Contributor

l5io commented Feb 6, 2026

🚀 Preview for commit e33a175 at: https://6985ed49e6962ec326ad1365--layer5.netlify.app

@rishiraj38
Copy link
Contributor Author

present it in monday's website meeting please

OK, but what was your initial thought about this? Does it look good?

@rishiraj38 rishiraj38 closed this Feb 6, 2026
@rishiraj38 rishiraj38 reopened this Feb 6, 2026
@rishiraj38
Copy link
Contributor Author

Like all components in the sistent, each has three sub-components: overview, guidance and code. I tried adding everything into a single index.mdx file but it became too large. It was too difficult to find and fix anything so I divided them into three separate files.

@l5io
Copy link
Contributor

l5io commented Feb 6, 2026

🚀 Preview for commit e33a175 at: https://698605f323fdac1a22f3038b--layer5.netlify.app

@Bhumikagarggg
Copy link

@rishiraj38 Thank you for your contribution! Let's discuss this during the website call tomorrow at 6:30 PM IST | 7 AM CST Add it as an agenda item to the meeting minutes, if you would 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/projects An issue relating to Layer5 initiatives (projects) area/site-config project/sistent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants