Skip to content

Commit 0b00d8e

Browse files
remove None from team role selection (#656)
Teams are required to have a Role, so None should not be an option.
1 parent b45b17a commit 0b00d8e

File tree

2 files changed

+1
-4
lines changed

2 files changed

+1
-4
lines changed

src/app/components/admin-app/roles-permissions-select/roles-permissions-select.component.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
(selectionChange)="updateRole($event.value)"
1818
placeholder="Role"
1919
>
20-
<mat-option value="">None</mat-option>
20+
<mat-option *ngIf="user" value="">None</mat-option>
2121
<mat-option *ngFor="let role of roles$ | async" [value]="role.id">
2222
{{ role.name }}
2323
</mat-option>

src/app/components/admin-app/roles-permissions-select/roles-permissions-select.component.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,13 +13,10 @@ import {
1313
Team,
1414
UserService,
1515
TeamService,
16-
RoleService,
1716
Permission,
18-
PermissionService,
1917
} from '../../../generated/player-api';
2018
import { TeamRolesService } from '../../../services/roles/team-roles.service';
2119
import { TeamPermissionsService } from '../../../services/permissions/team-permissions.service';
22-
import { forkJoin, tap } from 'rxjs';
2320
import { RolesService } from '../../../services/roles/roles.service';
2421

2522
export enum ObjectType {

0 commit comments

Comments
 (0)