Browse Source

Merge pull request #2252 from nocodb/fix/minor-corrections

fix: disable api docs access for viewer role
pull/2258/merge
navi 3 years ago committed by GitHub
parent
commit
eafa88983a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 22
      packages/nc-gui/components/ProjectTreeView.vue
  2. 3
      packages/nc-gui/helpers/rolePermissionsEE.js

22
packages/nc-gui/components/ProjectTreeView.vue

@ -681,16 +681,18 @@
</div>
<v-divider />
<div
v-t="['e:api-docs']"
class="caption pointer nc-docs pb-2 pl-5 pr-3 pt-2 d-flex align-center"
@click="openLink(apiLink)"
>
<v-icon color="brown" small class="mr-2">
mdi-open-in-new
</v-icon>
{{ $t('title.apiDocs') }}
</div>
<template v-if="_isUIAllowed('apiDocs')">
<div
v-t="['e:api-docs']"
class="caption pointer nc-docs pb-2 pl-5 pr-3 pt-2 d-flex align-center"
@click="openLink(apiLink)"
>
<v-icon color="brown" small class="mr-2">
mdi-open-in-new
</v-icon>
{{ $t('title.apiDocs') }}
</div>
</template>
<template v-if="_isUIAllowed('settings')">
<div class="pl-5 pr-3 d-flex align-center pb-2">

3
packages/nc-gui/helpers/rolePermissionsEE.js

@ -14,7 +14,8 @@ export default {
fieldsSync: true,
gridColUpdate: true,
filterSync: true,
csvImport: true
csvImport: true,
apiDocs: true
},
commenter: {
smartSheet: true,

Loading…
Cancel
Save