Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multi-tenant/teams settings seem to be missing #704

Closed
zperlo opened this issue Jun 22, 2021 · 6 comments
Closed

Multi-tenant/teams settings seem to be missing #704

zperlo opened this issue Jun 22, 2021 · 6 comments

Comments

@zperlo
Copy link

zperlo commented Jun 22, 2021

#701 is one part of this.

Additionally, I was previously using Client.Event.team to get the team on new message events, and it is no longer in the types. What happened to these values? Do you know the last version that supported this?

gz#12945

@shodgetts
Copy link

Agent comment from Seetha in Zendesk ticket #12945:

Hi,

Thanks for reaching out to us.
We will check this and come back to you soon.

Best,

°°°

@mahboubii
Copy link
Contributor

@zperlo Could you provide more details? I don't see any related changes to the type in the last few months.

@zperlo
Copy link
Author

zperlo commented Jun 23, 2021

@mahboubii Happy to, sorry I didn't have them in my first message. My team is upgrading chat in our app, as we are petty far behind. We were on version 2.1.2 and upgraded to 3.11.0. I had looked through the changelog, and I don't see any references to changes of these fields, which is why I was confused, given that we use them. Most of the changes seemed like they wouldn't affect us negatively, but we extensively use the teams api and need that in our events. I'm not sure why this wasn't listed as a breaking change/what version would be safe to upgrade to. Thanks for taking a look!

@zperlo
Copy link
Author

zperlo commented Jun 28, 2021

Any further info on this @mahboubii ?

@zperlo
Copy link
Author

zperlo commented Jul 12, 2021

Seems like #716 will fix this issue

@vishalnarkhede
Copy link
Contributor

This has been merged and will be released today

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants