ZoomNet icon indicating copy to clipboard operation
ZoomNet copied to clipboard

Site information returned from Phone endpoints may have changed

Open Jericho opened this issue 1 year ago • 2 comments

@joshrogers-pg Can you help investigating this situation? You submitted a PR back in January to add the ability to retrieve a phone call user profile and you even included a unit test to ensure we properly deserialize a sample JSON. The JSON you included clearly shows that "site_id" is a node in the JSON document. However, @snega1003 recently pointed out that documentation says something different. Evidently, they say that site id and name are grouped inside of a "site" json node. Can you help double check if the way you modeled the user profile (particularly the site information) is still accurate?

If it turns out that the way you model the data is still accurate, that will indicate that there's discrepancy how data is modeled between these two calls and @snega1003 should create his own model class.

Jericho avatar Apr 25 '24 14:04 Jericho

@Jericho @snega1003

I would have to verify locally, but this is running in production for us and we haven't been getting any serialization errors, so I'd imagine this is a discrepancy in the Zoom models. According to the documentation for the endpoint I implemented, retrieving the profile, the site_id is just a single node: https://developers.zoom.us/docs/api/rest/reference/phone/methods/#operation/phoneUser image

However, when you get a list of users as @snega1003 is doing, the site is an object: https://developers.zoom.us/docs/api/rest/reference/phone/methods/#operation/listPhoneUsers image

I would recommend creating his own model class. If you need me to verify beyond this, please let me know, there will be some turnaround time to that ask, but I'm happy to assist.

joshrogers-pg avatar Apr 25 '24 15:04 joshrogers-pg

If you don't mind, it would helpful if you could double-check

Jericho avatar Apr 25 '24 16:04 Jericho

@snega1003 implemented an alternate model for his scenario and it's working fine and I am going to assume that @joshrogers-pg 's model continues to work fine as well in his scenario.

Therefore, closing this issue.

Jericho avatar Jul 22 '24 14:07 Jericho