Skip to content

Commit 261daf8

Browse files
authored
fix: AppDeployment permissions granted to all if none selected by def… (#6935)
1 parent c899b17 commit 261daf8

File tree

4 files changed

+22
-3
lines changed

4 files changed

+22
-3
lines changed

.changeset/eight-planes-rule.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
---
2+
'hive': patch
3+
---
4+
5+
AppDeployment permissions granted to all if none selected by default in access token UI. Fix app
6+
deployments feature flag check

packages/services/api/src/modules/organization/resolvers/Organization.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
import { APP_DEPLOYMENTS_ENABLED } from '../../app-deployments/providers/app-deployments-enabled-token';
12
import { Session } from '../../auth/lib/authz';
23
import * as OrganizationAccessTokensPermissions from '../lib/organization-access-token-permissions';
34
import * as OrganizationMemberPermissions from '../lib/organization-member-permissions';
@@ -207,9 +208,11 @@ export const Organization: Pick<
207208
availableMemberPermissionGroups: () => {
208209
return OrganizationMemberPermissions.permissionGroups;
209210
},
210-
availableOrganizationAccessTokenPermissionGroups: async organization => {
211+
availableOrganizationAccessTokenPermissionGroups: async (organization, _, { injector }) => {
211212
const permissionGroups = OrganizationAccessTokensPermissions.permissionGroups;
212-
if (!organization.featureFlags.appDeployments) {
213+
const isAppDeploymentsEnabled =
214+
injector.get<boolean>(APP_DEPLOYMENTS_ENABLED) || organization.featureFlags.appDeployments;
215+
if (!isAppDeploymentsEnabled) {
213216
return permissionGroups.filter(p => p.id !== 'app-deployments');
214217
}
215218
return permissionGroups;

packages/web/app/src/components/organization/members/resource-selector.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -317,7 +317,7 @@ export function ResourceSelector(props: {
317317
targetId: item.id,
318318
targetSlug: item.slug,
319319
appDeployments: {
320-
mode: GraphQLSchema.ResourceAssignmentModeType.Granular,
320+
mode: GraphQLSchema.ResourceAssignmentModeType.All,
321321
appDeployments: [],
322322
},
323323
services: {

packages/web/app/src/components/organization/settings/access-tokens/create-access-token-sheet-content.tsx

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -324,6 +324,16 @@ export function CreateAccessTokenSheetContent(
324324
Granted on {permissionLevelToResourceName(group.level)}:
325325
</p>
326326
<ul className="flex list-none flex-wrap gap-1">
327+
{!resolvedResources[group.level]?.length && (
328+
<li>
329+
<Badge
330+
className="px-3 py-1 font-mono text-xs text-red-500"
331+
variant="outline"
332+
>
333+
No {group.level} selected.
334+
</Badge>
335+
</li>
336+
)}
327337
{resolvedResources[group.level].map(id => (
328338
<li key={id}>
329339
<Badge

0 commit comments

Comments
 (0)