Skip to content

Comments

add table component#6231

Closed
eyeaadil wants to merge 2 commits intolayer5io:masterfrom
eyeaadil:add-table-component
Closed

add table component#6231
eyeaadil wants to merge 2 commits intolayer5io:masterfrom
eyeaadil:add-table-component

Conversation

@eyeaadil
Copy link
Contributor

@eyeaadil eyeaadil commented Feb 10, 2025

Description

This PR fixes #5962
After
image
image

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Signed-off-by: eyeaadil <madil9227583@gmail.com>
@l5io
Copy link
Contributor

l5io commented Feb 10, 2025

🚀 Preview for commit ddc452a at: https://67aa6218f0fec017eb69909b--layer5.netlify.app

@mdkaifansari04
Copy link
Contributor

Good work @eyeaadil, but check why the lighthousecli is failing maybe there is some issue with build or git signing.

Try building it locally and re-push with changes, As it's hard to review without the preview site.

@eyeaadil
Copy link
Contributor Author

Good work @eyeaadil, but check why the lighthousecli is failing maybe there is some issue with build or git signing.

Try building it locally and re-push with changes, As it's hard to review without the preview site.

Okay I will fix this issue, currently I am busy in semester exams.

import { navigate } from "gatsby";
import { useLocation } from "@reach/router";
import {
Table,
Copy link
Member

Choose a reason for hiding this comment

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

don't use anything from ui, use Responsivetable component from sistent

Copy link
Member

Choose a reason for hiding this comment

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

see examples in meshery repo how tables being used...

@vishalvivekm
Copy link
Contributor

vishalvivekm commented Feb 17, 2025

@eyeaadil
Thank you for your contribution!
Let's discuss this during the website call on today at 6:30 PM IST

adding it as an agenda item to the meeting minutes.

@l5io
Copy link
Contributor

l5io commented Feb 17, 2025

🚀 Preview for commit 17f7508 at: https://67b305837be2aa85b0c2f55b--layer5.netlify.app

@Vidit-Kushwaha
Copy link
Contributor

@eyeaadil You have to add a components source to the components/index.js; you can take reference here.

@eyeaadil
Copy link
Contributor Author

@Vidit-Kushwaha I will fix the issue , currently my mid sem exams is start , so I am busy in that.

@sudhanshutech
Copy link
Member

@eyeaadil what's the status?

@stale
Copy link

stale bot commented May 1, 2025

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the issue/stale Issue has not had any activity for an extended period of time label May 1, 2025
@stale
Copy link

stale bot commented May 6, 2025

This issue is being automatically closed due to inactivity. However, you may choose to reopen this issue.

@stale stale bot closed this May 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

issue/stale Issue has not had any activity for an extended period of time project/sistent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Sistent] Add Table component to the sistent components page

6 participants