-
Notifications
You must be signed in to change notification settings - Fork 122
Duplicate #3
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
Comments
Hi, thanks a lot for contributing. This looks like a good improvement over our current iOS client. How would you feel taking ownership of this project and releasing your version as the next major version (1.0.0)? I would request you specify in the readme which tus version you target (v0.2.2) though. A next idea of improvement would be utilizing Travis to get some tests running, but that can be done later on. Let me know if you're interested and I'll make it happen. |
@kvz that sounds great. I'd like to help out. I will make the updates you mentioned. Thanks for responding so quickly. |
Alright I added you as a collaborator, please tag the current state as 0.0.1 and make your version the 1.0. Thanks for helping out! |
@kvz Sounds great, I'll take care of this today. Thanks and cheers! |
@kvz are you sure it was my username you added as a collaborator? It appears to either have not worked, or potentially someone else was added? I have some time today to get these changes made and get this repo sorted out. Thanks again. |
@kvz nvm, all is resolved. Disregard. |
Hi there,
I needed to use this client and had to make a few changes to get it to work. I also put it on Cocoapods. I changed the layout of the project to be more standard.
Anyway, I thought it worth offering to just transfer it to you guys if you'd like it. It's here: michaelavila/TUSKit, Let me know. As it stands the client you have in your repo does not work with any TUS servers. It has to do with a change in the headers, Final-Length should be Entity-Length. I've made that change already in my repo. And have verified that it works.
Let me know if you all want it. Take care.
The text was updated successfully, but these errors were encountered: