Skip to content
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

Feat: Added lazy loading for cards and loader for tables in admin panel successfully issue 438 #443

Merged
merged 4 commits into from
Jul 15, 2024

Conversation

MeetDOD
Copy link
Contributor

@MeetDOD MeetDOD commented Jul 14, 2024

Pull Request Resolves [#438 ]

Title: Added lazy loading for cards and loader for tables in admin panel successfully.

Description

  1. With help of react skeleton loader I had applied the lazy loading in Status cards of admin.
  2. With the help of react spinners I had applied the Circle ring loader in all the tables.
  3. The design is responsive.

Related Issues

Fixes #438

Changes Made

New features: Added lazy loading for cards and loader for tables in admin panel.

Screenshots

image

Video

Style.Share.Admin._.Dashboard._.-.Google.Chrome.2024-07-14.23-47-31.online-video-cutter.com.mp4

I certify that I have carried out the relevant code of conduct and provided the requisite screenshot for validation by submitting this pull request.

Thank You for this contribution.

Copy link

vercel bot commented Jul 14, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
style-share ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 15, 2024 8:12pm

Copy link

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

Copy link
Owner

@VaibhavArora314 VaibhavArora314 left a comment

Choose a reason for hiding this comment

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

Also add a loader for stats page since initially graphs are empty

@MeetDOD
Copy link
Contributor Author

MeetDOD commented Jul 15, 2024

Also add a loader for stats page since initially graphs are empty

Done 👍

Copy link
Collaborator

@Ultimateutkarsh11 Ultimateutkarsh11 left a comment

Choose a reason for hiding this comment

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

@MeetDOD kindly resolve the merge conflicts, rest looks good

@MeetDOD
Copy link
Contributor Author

MeetDOD commented Jul 15, 2024

@MeetDOD kindly resolve the merge conflicts, rest looks good

Done 👍

@Ultimateutkarsh11 Ultimateutkarsh11 merged commit 351ac6c into VaibhavArora314:main Jul 15, 2024
3 checks passed
Copy link

🎉 Your pull request has been successfully merged! 🎉 Thank you for your valuable contribution to our project. Your efforts are greatly appreciated. Feel free to reach out if you have any more contributions or if there's anything else we can assist you with. Keep up the fantastic work! 🚀

@Ultimateutkarsh11 Ultimateutkarsh11 added enhancement New feature or request level1 labels Jul 15, 2024
@MeetDOD
Copy link
Contributor Author

MeetDOD commented Jul 15, 2024

@Ultimateutkarsh11 see the efforts for finding this libraries and choosing the correct design and you are giving it LEVEL1 🗿 bruh ! Can you increase the level if possible ?

@MeetDOD
Copy link
Contributor Author

MeetDOD commented Jul 16, 2024

@Ultimateutkarsh11 see the efforts for finding this libraries and choosing the correct design and you are giving it LEVEL1 🗿 bruh ! Can you increase the level if possible ?

@VaibhavArora314

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request gssoc GSSOC level2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request]: Adding loader / lazy loading in the whole admin panel
4 participants