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

212 #213

Closed
wants to merge 7 commits into from
Closed

212 #213

wants to merge 7 commits into from

Conversation

kaustubhai
Copy link
Contributor

Problem

Github Issue Number: #212
Personal Overview was giving out the wrong information on the number of event and project created

Solution of problem

Modified the function and used proper methods to count the correct number

Type of Change

[x] Bugfix
[ ] New Feature
[ ] Development of UI/UX prototypes
[ ] Small refactor
[ ] Change in Documentation

Checklist

[x] My code follows the same style as the codebase
[ ] My Code change requires a change in documentation
[ ] I have updated the Readme accordingly
[x] I made PR against development branch
[x] I have run the test cases locally and it's passing.
[x] I have squashed my commits

@ksraj123
Copy link
Contributor

@kaustubhai Please remove package-lock.json from all of your PRs.

@kaustubhai
Copy link
Contributor Author

Sorry. Must have missed it in that one. Deleted

@ksraj123
Copy link
Contributor

@kaustubhai Why is this PR so massive now? 9799 deletions? Have you completely deleted the package-lock.json file? Just do not track changes to package-lock.json. Not include it in your PRs.

@kaustubhai
Copy link
Contributor Author

@ksraj123 So now should I copy the lock.json from main repo and send another PR?

@ksraj123
Copy link
Contributor

@kaustubhai I see that your PR uses commits from other PRs. This is not the best way to do things. I would suggest you to close all the PRs which depend upon changes made in your first PR and wait for that to be merged first.

@kaustubhai kaustubhai closed this Jan 18, 2021
@kaustubhai kaustubhai deleted the 212 branch January 18, 2021 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants