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

fix for add charge. start_date and end_date was not getting set #223

Closed
wants to merge 5 commits into from
Closed

fix for add charge. start_date and end_date was not getting set #223

wants to merge 5 commits into from

Conversation

imsagar
Copy link

@imsagar imsagar commented May 6, 2016

No description provided.

@drewish
Copy link

drewish commented May 6, 2016

This seems like it duplicates #220? Which one should we close?

@drewish
Copy link

drewish commented May 6, 2016

Oh actually seeing that this seems like it's a separate fix that was just branched off your other set of changes. Would you mind rebasing it on a clean copy of master and squashing down the repeated commits?

@imsagar
Copy link
Author

imsagar commented May 7, 2016

Sorry. I missed to check that. I have removed the other branch changes. This has only start and end date fix now.

@imsagar
Copy link
Author

imsagar commented May 25, 2016

Bump!

@drewish
Copy link

drewish commented May 25, 2016

Hi @imsagar could you please squash these down into a single commit?

@drewish
Copy link

drewish commented May 25, 2016

But if you'd rather not mess with the rebase I can take care of it in a separate PR.

@imsagar
Copy link
Author

imsagar commented May 26, 2016

Hey. I'll close this pull request. Have sent you clean one here #229

@imsagar imsagar closed this May 26, 2016
@bhelx bhelx added the V2 V2 Client label Mar 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
V2 V2 Client
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants