-
Notifications
You must be signed in to change notification settings - Fork 10.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
Regression: Fix marketplace releases tab crash bug #26162
Conversation
Fixed a bug where RC would crash because the marketplace releases tab was trying to display undefined data from manually installed apps.
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.
If they only provide one link, such as terms but not the privacy, I would vote we show whichever one we get back. Thoughts?
Makes sense, when I did this logic I believed both informations were mandatory. Will adapt it in a sec. |
Refactored the policies section to allow situations where only the tosLink or privacyLink where received from the back-end.
…tChat/Rocket.Chat into fix/marketplace-releases-tab-crash
Added missing translation keys for the Terms of use and Privacy Policy links on the AppSecurity component.
Fixed a problem where the lift of the useEndpointData from AppReleases to AppDetailsPage caused an unwanted toast to be fired. The hook was lifted to solve the releases tab appearing for manually installed apps, now it uses the marketplace field of the manually installed app to do this.
Proposed changes (including videos or screenshots)
Fixed a bug where RC would crash because the marketplace releases tab was trying to display undefined data from manually installed apps.
![app-releases-tab-crash-error](https://user-images.githubusercontent.com/43561537/177656489-325790d3-49e0-46c8-8ac2-1f74c6a309ad.gif)
Demo gif:
Issue(s)
Steps to test or reproduce
Further comments