Skip to content

fix issue for vacationBalance #11

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
Open
kaushikagrawal18 wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix issue for vacationBalance #11

kaushikagrawal18 wants to merge 2 commits into from

Conversation

@kaushikagrawal18
Copy link

@kaushikagrawal18 kaushikagrawal18 commented Oct 11, 2020

Hello @mukarramali, I have fixed your issue of negative vacation balance. Please, check this out and merge my pull request.

mukarramali
mukarramali suggested changes Oct 11, 2020
View reviewed changes
@mukarramali mukarramali added bug Something isn't working hacktoberfest This is to encourage people contribute in open source hacktoberfest-accepted Only valid PRs will be accepted in Hacktoberfest Medium People have little experience in the tech stack labels Oct 11, 2020
@mukarramali mukarramali linked an issue Oct 12, 2020 that may be closed by this pull request
Never allow vacation to go in negative #6
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mukarramali mukarramali Awaiting requested review from mukarramali

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

bug Something isn't working hacktoberfest This is to encourage people contribute in open source hacktoberfest-accepted Only valid PRs will be accepted in Hacktoberfest Medium People have little experience in the tech stack

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Never allow vacation to go in negative

2 participants

@kaushikagrawal18 @mukarramali