cloudinary / cloudinary-go

Cloudinary Golang package
MIT License
42 stars 21 forks source link

Possible incorrect param for a function #34

Closed stuartskelton closed 3 years ago

stuartskelton commented 3 years ago

Bug report for Cloudinary Go SDK

Before proceeding, please update to the latest version and test if the issue persists

Describe the bug in a sentence or two

func (u API) UpdateMetadata(ctx context.Context, params RenameParams) (UpdateMetadataResult, error) {

Takes a RenameParams I would have expected it take type UpdateMetadataParams struct {

Regards

Issue Type (Can be multiple)

Steps to reproduce

Code search,

Error screenshots

… if applicable

OS

Versions and Libraries (fill in the version numbers)

Repository

https://github.com/cloudinary/cloudinary-go/blob/3a4007e3a341968ff64e03b82789047bafd2a4c7/api/uploader/edit_asset.go#L109

wissam-khalili commented 3 years ago

Thank you for reporting this issue. An internal ticket was opened for this issue, though no ETA to share at the moment. We will update here with any progress.

wissam-khalili commented 3 years ago

Hi @stuartskelton I would like to update you the issue was resolved . Thanks, Wissam

stuartskelton commented 3 years ago

thank you