-
-
Notifications
You must be signed in to change notification settings - Fork 425
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
Pie chart generator added #3090
Conversation
The latest updates on your projects. Learn more about Vercel for Git βοΈ
|
Thank you for submitting your chaotic pull request! π We'll review it as soon as possible. In the meantime, please ensure that your changes align with our chaotic CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! π |
β Deploy Preview for getfinveda ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@pinakviramgama |
Dear Sir, @deepeshmlgupta i have not made changes in other files, just made changes in index.html and created piechart file, as i am working on another issue of testimonials.html as that file is not opening and only buffering so it might shown that there are changes. |
So, its better to check you commit before pushing it in your PR so that it should be clear. |
Sorry sir @deepeshmlgupta for that mistake, i will try not to repeat that, sir, i have updated the commit, please merge this PR. |
Sir @deepeshmlgupta , please merge this PR |
Dear @Sampadatiwari , please merge this PR |
Dear @Sampadatiwari , please merge this PR |
Dear @deepeshmlgupta , please merge this PR |
@pinakviramgama firstly you have to remove the unnecessary files that are not need for modification for this PR then only this PR will merge |
Dear @deepeshmlgupta , sir i tried my best to make previous changes revert , but sir i have not made any changes in other files and sir previous changes are not able to fetch, sorry sir for that. please sir guide me now what should i do. |
sir @deepeshmlgupta , it could be due to the single blank spaces that i by mistake have pressed in other files so changes might have reflected but sir i have not made any other changes any other files |
Sir @deepeshmlgupta , if you permit then can i delete those files from this PR which are not necessary so that if you merge this PR so that file changes won''t get reflected |
Deployment failed with the following error:
|
Sir @deepeshmlgupta , please guide me sir what should i do now, please sir guide me. |
you can delete those files that are not need so that it can easy for me to review this |
I am closing these PR so if you can delete those unrequited files than again reopen it |
π οΈ Fixes Issue
Fixes: #3053
π¨βπ» Description
What does this PR do?
Added new feature named pie chart generator which generates pie chart on basis of input from user of their savings, investments and expenses.
here's the output of it.
Home.-.FinVeda.-.Personal.-.Microsoft_.Edge.2024-11-09.11-25-41.1.1.mp4