That assumption didn't seem to work well with the current implementation of the upload feature, where it works in the tests and it doesn't in "real life" ☺
On 12/05/2015 13:44, Crístian Deives wrote:
> On 12-05-2015 13:11, Aline Manera wrote:
>> On 12/05/2015 12:46, Crístian Deives wrote:
>>> Shouldn't the UI be updated to re-enable file uploads now?
>>
>> I am working on UI right now.
>> As upload is current disabled on UI I don't see problems on merge the
>> backend first. Are you agree?
>
> Well, there's no problem in merging this code (as in "it won't break
> anything"), but there's no way for anyone to actually test a file
> upload and make sure the code works "in real life" (other than writing
> a test function...). I can't use the REST API via command line because
> this command requires passing a real object instead of JSON data in
> the variable "chunk", and there's no UI to use it as well. So only
> with the UI I'll be able to actually test if the implementation works.
Well, we have a specific test case to use file upload. So if the test
passes...