-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added the List component to the sistent components page and updated the content.js file. #6219
base: master
Are you sure you want to change the base?
Conversation
…he content.js Signed-off-by: Roshan Goswami <csengineer23@gmail.com>
🚀 Preview for commit 5c557f3 at: https://679f638e6e5e0b86aee4c955--layer5.netlify.app |
@csengineer1990 adding it as an agenda item to the meeting minutes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not require to create a separate file for code-block.js you can directly import, so delete this file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you include example for the List controls like checkbox?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have provided few feedback can you look @csengineer1990
import React, { useState } from "react"; | ||
import Code from "../../../../../components/CodeBlock"; | ||
|
||
export const CodeBlock = ({ name, code }) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use existing codeblock and don't create a new one please
Signed-off-by: Roshan Goswami <csengineer23@gmail.com>
…oned in the previous request. Signed-off-by: Roshan Goswami <csengineer23@gmail.com>
Made the requested changes by adding examples of List control with checkboxes and removing code-block.js. Apologies for the delayed update. If any further changes are needed, please let me know. |
🚀 Preview for commit 2a9bd74 at: https://67b08c32b7f3d319c41cfb1e--layer5.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks Good to me
@csengineer1990 please have a look at the merge conflicts here. |
Yes, I will surely resolve the merge conflicts as soon as possible. Thanks for bringing it to my attention! |
Signed-off-by: Roshan Goswami <csengineer23@gmail.com>
Signed-off-by: Roshan <68468678+csengineer1990@users.noreply.github.com>
🚀 Preview for commit fefa978 at: https://67b954e04b59d7b097203a3b--layer5.netlify.app |
I have resolved the merge conflict. Please review the commit and let me know if there are any issues so that I can address them as soon as possible. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved 👍
Add this as an agenda item to the meeting minutes.
Description
This PR addresses issue #5951 by adding the List component to the Sistent Components page. The necessary content and documentation for the List component have been included, along with updates to the relevant files to ensure consistency.
This PR fixes #5951
Notes for Reviewers
I have made the following changes:
I have made the required changes as requested in the previous reviews and aligned them with the issue to ensure a proper resolution.
I would appreciate your review of these updates. Please let me know if any further refinements or additional changes are needed. Thank you!
Signed commits