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

Issue #67 - Add support for & in attachments #68

Merged
merged 4 commits into from
Aug 17, 2017

Conversation

jordanwalsh23
Copy link
Contributor

An issue was identified around signature generation. The fix was to do this prior to downloading the attachment:

attachment.Url = attachment.Url.replace(/%26/i, "&");

This PR makes this change and modifies some of the attachment tests to make them more robust.

@jordanwalsh23
Copy link
Contributor Author

Fix for #67

@jordanwalsh23 jordanwalsh23 merged commit cda9b3a into XeroAPI:master Aug 17, 2017
@jordanwalsh23 jordanwalsh23 deleted the issue_67 branch August 17, 2017 00:34
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.

1 participant