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

Update the API to support proper classes, remove raw maps, class responses #239

Open
bjornharvold opened this issue May 22, 2021 · 3 comments
Assignees

Comments

@bjornharvold
Copy link

Feature request for Cloudinary Java SDK

I think it's safe to say that most Java users today use >= JDK 8. I would be great to have a Cloudinary API you could use and know exactly what you need to send and exactly what you are going to get back without having to read the docs on what JSON responses return or what you could pass for the "Object" parameter in a method like upload().

Adding support for uploading InputStream and byte[] would also be a great addition.

Explain your use case

Asking for a self-explanatory API

Describe the problem you’re trying to solve

Not spending so much time going through the documentation on what raw Maps include.

Do you have a proposed solution?

  • Instead of passing Objects, create convenience methods that take self-explanatory classes.
  • Instead of raw maps, pass class objects with the proper / available accessors.

In a nutshell, make this API more friendly. Currently, every time I have to work with the Cloudinary Java API, I have to start from scratch and read the docs over again to see what's available to me. Look at Stripe's Java API for an example of an incredibly well-formed API.

@michalkcloudinay michalkcloudinay self-assigned this May 24, 2021
@michalkcloudinay
Copy link
Contributor

Thanks @bjornharvold for your suggestion. We will open a request internally to support this and will update here with any new insights.

@KyriacosP
Copy link

+1
Any progress on this? Having raw Maps and Objects everywhere is proving very difficult to work with.

@aksjoshi1
Copy link
Collaborator

After careful review, our team has decided not to go ahead with this request as it requires a breaking change.

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

No branches or pull requests

4 participants