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-#12: Added Statistics Section on home page #236

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Ashu0216
Copy link

Description

Added a statistics page which shows the number od notes , students and subjects that the platform has.
While the data is being from the database , a loader will be shown
Added the menu option in navbar to access the statistics page
The page is completely responsive.

Resolves: [#12 ]

Checklist

Before submitting this pull request, kindly verify that the ensuing checkpoints have been reached.

  • Have you adhered to the repository's defined coding convention rules?
  • Have you sent a message along with the result or response?
  • Have you used the try-catch technique?

Screenshots

Screenshot 2024-05-22 193737

Screenshot 2024-05-22 193843

Screenshot 2024-05-22 193900

Additional Notes/Comments

This is a great feature to showcase the use of 'ExamTime' project by students. I am open to any suggestion or changes.


I certify that I have carried out the relevant checks and provided the requisite screenshot for validation by submitting this pull request.
I appreciate your contribution.

Copy link

vercel bot commented May 25, 2024

@Ashu0216 is attempting to deploy a commit to the swapnilsingh99's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Great job, @Ashu0216! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project.

We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

@Ashu0216
Copy link
Author

Hi @hereisSwapnil , kindly review the pull request and let me know if there any suggestions or changes.
Really enjoyed working on the issue😊.

@hereisSwapnil
Copy link
Owner

@Ashu0216 Please resolve conflicts
and also the page look kind of empty, so what do you suggest further?

@Ashu0216
Copy link
Author

Ashu0216 commented Jun 3, 2024

Hi @hereisSwapnil , I have resolved the conflicts.
I was busy in my exams, Thank you for your patience.
Can you please review and finish the pull request.

@Ashu0216
Copy link
Author

Ashu0216 commented Jun 4, 2024

@hereisSwapnil Please assign the same level (level 2) on the PR as granted on the issue #12 . Thank you

@Ashu0216
Copy link
Author

Ashu0216 commented Jun 6, 2024

Hi @hereisSwapnil , can you please review and approve the PR.

@Ashu0216
Copy link
Author

@hereisSwapnil pls review the PR

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

Successfully merging this pull request may close these issues.

2 participants