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

Add missing headers in client #95

Merged
merged 2 commits into from
Oct 24, 2018
Merged

Add missing headers in client #95

merged 2 commits into from
Oct 24, 2018

Conversation

ankitpokhrel
Copy link
Owner

@ankitpokhrel ankitpokhrel commented Oct 23, 2018

Related: #91

  • Add missing headers in client
  • Verify against protocol doc
  • Fix tests
  • Regression test

@ankitpokhrel ankitpokhrel self-assigned this Oct 23, 2018
@ankitpokhrel ankitpokhrel added the client Tus Client label Oct 23, 2018
Copy link
Collaborator

@samundra samundra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cheers 🎉

@ankitpokhrel ankitpokhrel merged commit 2bacfe3 into master Oct 24, 2018
@ankitpokhrel ankitpokhrel deleted the issue-91 branch October 24, 2018 03:40
@erunion
Copy link

erunion commented Nov 15, 2018

@ankitpokhrel Any plans to tag this change to a new release?

@ankitpokhrel
Copy link
Owner Author

@erunion Yes, will tag a release this weekend or early next week. You can use dev-master branch until then.

@ankitpokhrel
Copy link
Owner Author

@erunion I tagged new version v0.1.0.

Just a heads-up, client implementation will change a bit in upcoming release and it might break your integration. You can review the changes here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants