-
Notifications
You must be signed in to change notification settings - Fork 11.1k
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
[FIX] Remove deprecated permissions api endpoint #12285
[FIX] Remove deprecated permissions api endpoint #12285
Conversation
@RocketChat/android @RocketChat/ios I don't think you guys are using this. But as typical heads up. Since we did schedule to be deprecated |
@geekgonecrazy @kaiiiiiiiii Wow, thanks for the heads up! Yes, we are using it! What API should we use? |
@rafaelks you should use |
Lets no accidentally merge this yet
I guess we'll need to wait few releases now... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's hold on until 0.73 at least? Mobile apps needs to be updated.
@rafaelks yes of course, but the endpoint has been deprecated since |
@sampaiodiego If |
Guys, can we hold on this change to |
@rafaelks I needed to change the endpoint again, so I ended up deprecating |
@MarcosSpessatto Got it! OK, let's plan to deprecate the APIs 8~12 versions after we create the deprecation message, because currently the iOS and Android apps are supporting 10 version older (0.62+) of the back-end and would be really painful to start supporting all these deprecations right now, unless there's a serious performance issue or any other reason that would really require this update. |
Cheers,
Kai