Changeset View
Changeset View
Standalone View
Standalone View
src/resources/vue/Rooms.vue
| Show All 35 Lines | export default { | ||||
| data() { | data() { | ||||
| return { | return { | ||||
| rooms: [], | rooms: [], | ||||
| href: '', | href: '', | ||||
| roomRoute: '' | roomRoute: '' | ||||
| } | } | ||||
| }, | }, | ||||
| mounted() { | mounted() { | ||||
| if (!this.$root.hasSKU('meet')) { | if (!this.$root.hasSKU('meet') || this.$root.isDegraded()) { | ||||
mollekopf: How about moving the isDegraded check to hasSKU instead?
Or even more centrally in… | |||||
Done Inline ActionsI would agree that we could do something better in statusInfo, but I don't think we have to do this here. hasSKU + isDegraded is used in one place. In other places we have to just hide some elements like Create buttons. So, it would not give us much. When we decide to do something more sophisticated with the UI we might get to that. Diff has been rebased. machniak: I would agree that we could do something better in statusInfo, but I don't think we have to do… | |||||
| this.$root.errorPage(403) | this.$root.errorPage(403) | ||||
| return | return | ||||
| } | } | ||||
| this.$root.startLoading() | this.$root.startLoading() | ||||
| axios.get('/api/v4/openvidu/rooms') | axios.get('/api/v4/openvidu/rooms') | ||||
| .then(response => { | .then(response => { | ||||
| Show All 12 Lines | |||||
How about moving the isDegraded check to hasSKU instead?
Or even more centrally in UsersController::statusInfo, which seems to be where we decide who can view what.
Same goes for all the checks above, so something different than hasSKU is perhaps required:
hasCapability("identifier") or so.