Cloudex.upload returns list instead of map about cloudex HOT 5 CLOSED

Done :) 1.0.0 is now published, returning {:ok, %Cloudex.UploadedImage{}} for a single file / url and a list of those when uploading multiple.

from cloudex.

Hi,
I agree that this is an anomaly indeed. Ideally it should return a tuple if only one file is uploaded and a list of tuples when multiple where uploaded.

I'd like to change this behaviour but do not really know what the best way would be to make this change with the least amount of disruption for the current users.

One possibility would be to do a major version increase to make this change, with a clear note of the breaking change, and the usual annotation in the changelog. I'm also open for more suggestions on how to properly make this change. Pinging some contributors to see if they might have some input as well on this :) @nathanl @Tyler-pierce @pauloancheta @nbancajas @sudostack @remiq

Thanks in advance, Gerard

from cloudex.

I'd vote for "make the change and do a major version bump, explaining in the changelog." I see that Cloudex isn't at 1.0 yet, but I think it could be. Nothing is ever complete, but what it does, it does well.

from cloudex.

+1 major version change to notify existing users of the breaking interface. Curious, as I am the first to mention the discrepancy, are most users passing a list. Wonder what percentage of users are passing a single, non-list input to the method.

from cloudex.

+1 on major change and explain in the changelog. And as @billc have mentioned, there might not be a lot of users a non-list input to the method. Disruption to other users would be minimal.

from cloudex.