Client upgraded permissions sub super #147
Conversation
Port Forwarding feature
Beta export option
Server suspend text
Option to send welcome emails to sub/super clients
…client-upgraded-permissions-sub-super
app/user/user.sidebar.js
Outdated
| function NavConfig(Auth, NavProvider, Permission) { | ||
| //Need to import Auth and Permission here. Why doesn't this work? in install.nav.config.js and search.tab.js | ||
| //this is used in seemingly the exact same way. | ||
| var group = NavProvider.group('user', { |
There was a problem hiding this comment.
why doesnt this work? the other times auth is used it appears to be the exact same.
There was a problem hiding this comment.
It's because install.nav.config.js uses .run() instead of .config() where line 21 is in this file.
app/user/user.sidebar.js
Outdated
| function NavConfig(Auth, NavProvider, Permission) { | ||
| //Need to import Auth and Permission here. Why doesn't this work? in install.nav.config.js and search.tab.js | ||
| //this is used in seemingly the exact same way. | ||
| var group = NavProvider.group('user', { |
There was a problem hiding this comment.
It's because install.nav.config.js uses .run() instead of .config() where line 21 is in this file.
| @@ -62,5 +63,9 @@ | |||
| function assignClientModal() { | |||
| return ServerAssignModal.client([assign.server], assign.server.access.sub); | |||
| } | |||
|
|
|||
| function hasSubSuperAccess() { | |||
There was a problem hiding this comment.
since this is a function, you would need to use assign.client.subSuperAccess() instead of assign.client.subSuperAccess to get the returned value from the function.
We'll need this implemented for real, what's the plan here? We would need to fetch this off of the authenticated user's account details. Since we'd be using this in multiple places, we may need to add another service to get & cache this information, or maybe we can cache it with the user details (like user name)?
| text: "Super Clients", | ||
| sref: "app.user.client.super.list" | ||
| }); | ||
| function UserSidebarShowWithPermissions(Auth, Select /*, Permission*/) { |
There was a problem hiding this comment.
permission can't be imported because this is the client side. Clients don't have permissions - only Admins & Integrations do. We need a way of getting this information off of the client account, e.g. GET /client/{authed ID}.
No description provided.