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 validate url path #2

Merged
merged 4 commits into from Sep 18, 2013
Merged

Fix validate url path #2

merged 4 commits into from Sep 18, 2013

Conversation

rlubbat
Copy link
Contributor

@rlubbat rlubbat commented Aug 9, 2013

Change validate url path to /api/push/validate/

Change validate url path to /api/push/validate/
Added note about modification to fix validate API path.
Set version to 0.1.3.XOOM and add Xoom information.
@mhooge
Copy link
Contributor

mhooge commented Aug 17, 2013

My apologies for the late reply, we're still ironing out email filters (We've a have a lot of repos).

Thank you for the fix. If you wouldn't mind filling out our Contribution agreement, I'll merge in your changes.

http://urbanairship.com/legal/contribution-agreement

Cheers.

@rlubbat
Copy link
Contributor Author

rlubbat commented Aug 17, 2013

Done. Please note I missed one instance of /api/validate in a unit test. You can just search for it and replace with /api/push/validate.

On Aug 17, 2013, at 6:56 AM, Matt Hooge notifications@github.com wrote:

My apologies for the late reply, we're still ironing out email filters (We've a have a lot of repos).

Thank you for the fix. If you wouldn't mind filling out our Contribution agreement, I'll merge in your changes.

http://urbanairship.com/legal/contribution-agreement

Cheers.


Reply to this email directly or view it on GitHub.

@mhooge
Copy link
Contributor

mhooge commented Aug 18, 2013

Thank you. I'll merge in your work and fix the remaining test.

@mhooge mhooge merged commit aad0bb8 into urbanairship:master Sep 18, 2013
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