Spelling Mistakes in Android SDK

I noticed that the us.zoom.sdk.InMeetingServiceListener interface includes the method name onMeetingNeedColseOtherMeeting. Should this actually be onMeetingNeedCloseOtherMeeting?

The sample app also has an argument named isOrignalHost instead of isOriginalHost here: https://github.com/zoom/zoom-sdk-android/blob/f4b5d088a6b388351a73aa0c58a539b4b3484951/mobilertc-android-studio/sample/src/main/java/us/zoom/sdksample/inmeetingfunction/customizedmeetingui/SimpleInMeetingListener.java#L144

Hi @roger,

Thanks for the post and really appreciate that you pointed out the spellings. I will forward this to the engineering team.

Thanks!

Thanks Carson!

I noticed the following additional issues. Would you be able to forward these to the engineering team as well?

  1. The Javadoc for ZoomSDKl#getInMeetingService says “It will return null if the SDK has not been initialized successfully.” However, it seems to return null when the SDK has been initialized successfully, but the user is not currently in a meeting. Should the Javadoc explain this?
  2. MobileRTCVideoViewManager has no destroy method. But the following two pages indicate that it does: https://marketplace.zoom.us/docs/sdk/native-sdks/android/mastering-zoom-sdk/in-meeting-function/customized-meeting-ui/other#13-when-leaving-or-ending-a-meeting and https://marketplace.zoom.us/docs/sdk/native-sdks/android/mastering-zoom-sdk/in-meeting-function/customized-meeting-ui/overview#2-create-custom-meeting-ui
  3. This page says “destory” instead of destroy in two places. It has the line “viedeoViewMgr .destory();” in the code sample with a space. But it seems like that space should not be there.
  4. How should I “Ensure to destory video units before meeting activity finish?” The docs say to use MobileRTCVideoViewManager#destory but this method does not exist.