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.

Sign up for GitHub

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

Jump to bottom

Feat: Added graphs(Line, Bar, Bubble, Scatter) for admin successfully issue 367 #410

Closed
wants to merge 5 commits into from
Closed

Feat: Added graphs(Line, Bar, Bubble, Scatter) for admin successfully issue 367 #410

wants to merge 5 commits into from

Conversation

MeetDOD
Copy link
Contributor

@MeetDOD MeetDOD commented Jul 10, 2024

Pull Request Resolves[#367 ]

Title: Added graphs(Line, Bar, Bubble, Scatter) for admin successfully.

Description

  1. In admin backend, I created a controller called getgraphsstatus and it's routes also this api give the created date of any instance in style share i.e users, posts, comments, favorities, contactmessages and reactions.
  2. In admin frontend, I created a Graph page also updated the sidebar and router also.
  3. In Graph page i had used chartjs and react-chartjs-2 and I am fetching the data from the api and converting it into four graph format for admin.
  4. The design is responsive in all devices.

Related Issues

Fixes #367

Changes Made

New feature: Added graphs(Line, Bar, Bubble, Scatter) for admin.

Screenshots

image

image

image

image

image

image

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.

@vercel Vercel
Copy link

vercel bot commented Jul 10, 2024 •
edited
Loading

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 11, 2024 3:12pm

@github-actions GitHub Actions
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! 😊

VaibhavArora314
VaibhavArora314 requested changes Jul 11, 2024
View reviewed changes
Copy link
Owner

@VaibhavArora314 VaibhavArora314 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

image
image

@MeetDOD
Copy link
Contributor Author

MeetDOD commented Jul 11, 2024

image image

@VaibhavArora314 in my local it is fetching in deployment why it is not rendering it ?

@VaibhavArora314
Copy link
Owner

@MeetDOD
image

@VaibhavArora314
Copy link
Owner

the old records do not have the createdAt field that's why this error is coming I guess

@MeetDOD
Copy link
Contributor Author

MeetDOD commented Jul 11, 2024

Ya I got it so what to do now ?

@MeetDOD
Copy link
Contributor Author

MeetDOD commented Jul 11, 2024

can we just create new data or something like you delete the reactions from the database that will not heart anything or else we use other approach if you have !

@VaibhavArora314
Copy link
Owner

can we just create new data or something like you delete the reactions from the database that will not heart anything or else we use other approach if you have !

I will update the database, give me some time

VaibhavArora314
VaibhavArora314 requested changes Jul 11, 2024
View reviewed changes
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.

Is it possible to add a msg below each graph or label the x-y axis to better understand what the graph is about?

@MeetDOD
Copy link
Contributor Author

MeetDOD commented Jul 11, 2024

Is it possible to add a msg below each graph or label the x-y axis to better understand what the graph is about?

@VaibhavArora314 I label the graphs at x and y axis respectively, this library has built in feature 😄

@VaibhavArora314
Copy link
Owner

image

what are the two charts below for?

@MeetDOD
Copy link
Contributor Author

MeetDOD commented Jul 12, 2024

image

what are the two charts below for?

So, every api like user,post,reaction all of them i created four graphs, line,bar,bubble,scatter respectively.
so the above two are:
line | bar
bubble | scatter

like this 👍

@VaibhavArora314
Copy link
Owner

image
what are the two charts below for?

So, every api like user,post,reaction all of them i created four graphs, line,bar,bubble,scatter respectively. so the above two are: line | bar bubble | scatter

like this 👍

I don't think bubble and scatter graphs have much use in most cases with respect to time for our application. Can you please remove them, rest is looking good.

@vercel Vercel
Copy link

vercel bot commented Jul 12, 2024

Deployment failed with the following error:

The provided GitHub repository does not contain the requested branch or commit reference. Please ensure the repository is not empty.

@MeetDOD
Copy link
Contributor Author

MeetDOD commented Jul 12, 2024

image
what are the two charts below for?

So, every api like user,post,reaction all of them i created four graphs, line,bar,bubble,scatter respectively. so the above two are: line | bar bubble | scatter
like this 👍

I don't think bubble and scatter graphs have much use in most cases with respect to time for our application. Can you please remove them, rest is looking good.

It will look nice when the data will be populated also it will be easy to admin to visualize, But never mind i removed it @VaibhavArora314 👍

@MeetDOD
Copy link
Contributor Author

MeetDOD commented Jul 12, 2024

Deployment failed with the following error:

The provided GitHub repository does not contain the requested branch or commit reference. Please ensure the repository is not empty.

@VaibhavArora314 what is this ?

@VaibhavArora314
Copy link
Owner

VaibhavArora314 commented Jul 12, 2024 •
edited
Loading

Deployment failed with the following error:

The provided GitHub repository does not contain the requested branch or commit reference. Please ensure the repository is not empty.

@VaibhavArora314 what is this ?

Can't find anything in vercel logs and the website is also not updated. It's probably a vercel error, let me look into it.

@VaibhavArora314
Copy link
Owner

Most probably you will have to create a new pull request as vercel is not detecting up changes in this PR.

@MeetDOD
Copy link
Contributor Author

MeetDOD commented Jul 12, 2024

Most probably you will have to create a new pull request as vercel is not detecting up changes in this PR.

🗿

@MeetDOD
Copy link
Contributor Author

MeetDOD commented Jul 12, 2024

Most probably you will have to create a new pull request as vercel is not detecting up changes in this PR.

Ok I will do that give me a minute

@MeetDOD
Copy link
Contributor Author

MeetDOD commented Jul 12, 2024

Now it is checking/working when i cratead a new PR 🗿

@VaibhavArora314
Copy link
Owner

Now it is checking/working when i cratead a new PR 🗿

Lmao, anyways closing this now since #425 is created

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@VaibhavArora314 VaibhavArora314 VaibhavArora314 requested changes

@akbatra567 akbatra567 Awaiting requested review from akbatra567

@Ultimateutkarsh11 Ultimateutkarsh11 Awaiting requested review from Ultimateutkarsh11

Assignees

@MeetDOD MeetDOD

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feature Request]: Adding separate graphs page
2 participants
@MeetDOD @VaibhavArora314