Add support for `PlaybackUrl` in `VideoUploadResult`

[const-cloudinary]

Feature request for Cloudinary .NET SDK

When uploading a video file the result contains playback_url field which is currently not exposed in Cloudinary .NET SDK.

Do you have a proposed solution?

Add PlaybackUrl field in VideoUploadResult

[Suraj3240]

@const-cloudinary Where should the tests be added for this if the playback_url field is empty?

[AnshShah77]

@const-cloudinary Can u assign this to me please?

[colbyfayock]

@Suraj3240 are you already working on this?

[Suraj3240]

@Suraj3240 are you already working on this?

I have added the playbackurl field. I wanted to know where to add the ckeck for empty playbackurl thing?

[colbyfayock]

@const-cloudinary should be able to answer that within the next few days (it's the weekend) but making sure someone else didn't start working on it if you were already :)

[Suraj3240]

So will it be assigned to me or to @AnshShah77 ?

[colbyfayock]

you said you were already working on it, so you! :) we just ask for a little patience for the assignment and for someone from the SDK team (not me) to respond to your question above. thanks

[Suraj3240]

Ok great :) Thanks Sorry @AnshShah77 🙈 @colbyfayock Ansh is my friend only :))

[AnshShah77]

np @Suraj3240 , all the best!!

[const-cloudinary]

@Suraj3240 Regarding your question, this field should not be empty for any video. Having said that, if the field is empty - it should not be a problem, it will be left as an empty string. Adding a test where playback url is not empty should be sufficient.

[Suraj3240]

So in which file should I add a test for this?

[const-cloudinary]

@Suraj3240 this looks like a good candidate: https://github.com/cloudinary/CloudinaryDotNet/blob/a3df716e77aa81db98aea61002f3b881d6a7de9a/CloudinaryDotNet.IntegrationTests/UploadApi/UploadMethodsTest.cs#L203