search

dji-sdk / Mobile-SDK-Android-V5

MSDK V5 Sample
Other
267 stars 143 forks source link

Release 5.9.1 - Breaking Change #365

Closed antonymarion closed 3 months ago

antonymarion commented 3 months ago

Hi Dji

On previous releases, availableCameraListData used to be :

image

it is now a simple String from following Enum:

LEFT_OR_MAIN, RIGHT, UP, FPV, UP_TYPE_C, AGGREGATION, UNKNOWN

As we are using your SDK to trigger some behaviour based on it, could you please take care to mention this breaking change in the release Note, and to avoid this kind of breaking change to ease usability and maintenance of the code

Regards,

dji-dev commented 3 months ago

Agent comment from yating.liao in Zendesk ticket #109466:

I'm having trouble understanding your question. Are you asking if there are differences in AvailableCameraUpdatedListener between versions 5.8 and 5.9?

It's important to note that AvailableCameraUpdatedListener is a new listener type introduced in version 5.8. Prior to version 5.8, StreamSourceListener was used, but it is now deprecated.

°°°

antonymarion commented 3 months ago

the format of the object availableCameraListData has changed

dji-lyt commented 3 months ago

CameraStreamListVM has not been modified since version 5.8. Also, the definition of availableCameraListData has not changed. What version were you using before?

antonymarion commented 3 months ago

One of our developpers has changed your source code that's why ...

Sorry for the disturbance!

dji-dev commented 3 months ago

Agent comment from yating.liao in Zendesk ticket #109466:

Don't worry about it. As long as you have found the cause of the issue, it's fine.

°°°