[Sistent] feat: refactoring and implementing dynamic routing for Sistent components.#7408
Conversation
github-actions
bot
added
area/projects
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
force-pushed
the
feat/sistent-refactor-mdx
branch
from
860abe1 to
b7bc06b
Compare
|
@Rajesh-Nagarajan-11 Could you please review and give some feedback? |
|
🚀 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>
|
@Rajesh-Nagarajan-11 Could you please review this? Is this the correct way? |
|
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. 😅 |
|
present it in monday's website meeting please |
|
🚀 Preview for commit e33a175 at: https://6985ed49e6962ec326ad1365--layer5.netlify.app |
OK, but what was your initial thought about this? Does it look good? |
|
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. |
|
🚀 Preview for commit e33a175 at: https://698605f323fdac1a22f3038b--layer5.netlify.app |
|
@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 🙂 |
|
🚀 Preview for commit 83035c1 at: https://698ece6c6623f82919a30f8f--layer5.netlify.app |
Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
rishiraj38
force-pushed
the
feat/sistent-refactor-mdx
branch
from
4d638b7 to
42b3496
Compare
…or-mdx # Please enter a commit message to explain why this merge is necessary, # especially if it merges an updated upstream into a topic branch. # # Lines starting with '#' will be ignored, and an empty message aborts # the commit.
|
🚀 Preview for commit 43f7ff8 at: https://69932c5945919983b1785c12--layer5.netlify.app |
Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
|
@Rajesh-Nagarajan-11 I understand I couldn’t explain everything because of a sore throat which makes speaking difficult. This will make reviewing challenging. Could you please review one component, the accordion, and provide some feedback? This will help me adjust all other components similarly. |
|
Did you use AI to migrate contents ? |
|
If so no issues , but please double check the doc contents are replica , becuz AI might change the content of documention |
|
🚀 Preview for commit 5c3ca69 at: https://69933d87b32a291bd40f03b8--layer5.netlify.app |
|
🚀 Preview for commit 1516d98 at: https://69b19d709ca68c0dd9fa2c9f--layer5.netlify.app |
|
LGTM!! |
Signed-off-by: Rajesh Nagarajan <139469505+Rajesh-Nagarajan-11@users.noreply.github.com>
Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
|
🚀 Preview for commit 7fd191f at: https://69b5aba67cf20979573e2a41--layer5.netlify.app |
Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
|
@Rajesh-Nagarajan-11 Please review . |
|
🚀 Preview for commit 080eb92 at: https://69b8362d0687226e227d86f5--layer5.netlify.app |
|
🚀 Preview for commit 76192fd at: https://69b8d9d00aad565c6855014d--layer5.netlify.app |
|
🚀 Preview for commit 2e94b26 at: https://69b991f8498ea3146fed41d0--layer5.netlify.app |
|
🚀 Preview for commit 2e906b7 at: https://69ba435066e2eaff7f7d2b60--layer5.netlify.app |
YASHMAHAKAL
left a comment
YASHMAHAKAL
left a comment
There was a problem hiding this comment.
Looks Good @rishiraj38 !
|
Thanks for working on this @rishiraj38 |
rishiraj38
changed the title
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.
This PR also Fixes #7426 & #7451
Components Refactored:
Signed commits