Skip to content

Commit 33d4cf3

Browse files
author
Natalia Kupriianova
committed
fix: [MULTIDIRECTORY-1272] Remove DNS functions from the front
1 parent 5d5a80e commit 33d4cf3

12 files changed

Lines changed: 80 additions & 99 deletions

File tree

projects/multidirectory-app/src/app/features/dhcp/tabs/reservation/reservation.component.ts

Lines changed: 19 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -13,14 +13,7 @@ import { catchError } from 'rxjs';
1313
selector: 'reservation',
1414
templateUrl: './reservation.component.html',
1515
styleUrls: ['./reservation.component.scss'],
16-
imports: [
17-
DatagridComponent,
18-
ButtonComponent,
19-
TranslocoPipe,
20-
ButtonComponent,
21-
ButtonComponent,
22-
ButtonComponent,
23-
],
16+
imports: [DatagridComponent, ButtonComponent, TranslocoPipe, ButtonComponent, ButtonComponent, ButtonComponent],
2417
})
2518
export default class DhcpReservationComponent implements OnInit {
2619
private dialogService = inject(DialogService);
@@ -59,23 +52,22 @@ export default class DhcpReservationComponent implements OnInit {
5952
}
6053

6154
delete() {
62-
const selectedReservationExist =
63-
this.selectedReservation && Object.keys(this.selectedReservation).length > 0;
55+
const selectedReservationExist = this.selectedReservation && Object.keys(this.selectedReservation).length > 0;
6456
selectedReservationExist &&
65-
this.dhcp
66-
.deleteDhcpReservation({
67-
ip_address: this.selectedReservation?.ip_address ?? '',
68-
mac_address: this.selectedReservation?.mac_address ?? '',
69-
subnet_id: this.subnet.id,
70-
})
71-
.pipe(
72-
catchError((err) => {
73-
throw err;
74-
}),
75-
)
76-
.subscribe(() => {
77-
this.dhcp.getReservationsList(this.subnet.id);
78-
});
57+
this.dhcp
58+
.deleteDhcpReservation({
59+
ip_address: this.selectedReservation?.ip_address ?? '',
60+
mac_address: this.selectedReservation?.mac_address ?? '',
61+
subnet_id: this.subnet.id,
62+
})
63+
.pipe(
64+
catchError((err) => {
65+
throw err;
66+
}),
67+
)
68+
.subscribe(() => {
69+
this.dhcp.getReservationsList(this.subnet.id);
70+
});
7971
}
8072

8173
add() {
@@ -103,7 +95,7 @@ export default class DhcpReservationComponent implements OnInit {
10395
subnet_id: this.subnet.id,
10496
ip_address: ip,
10597
mac_address: mac,
106-
hostname: host
98+
hostname: host,
10799
};
108100
}
109101

@@ -113,8 +105,8 @@ export default class DhcpReservationComponent implements OnInit {
113105
component: DhcpAddReservationComponent,
114106
dialogConfig: {
115107
width: '550px',
116-
data: new ReservationDataWrapper(data, reservationExists)
117-
}
108+
data: new ReservationDataWrapper(data, reservationExists),
109+
},
118110
})
119111
.closed.subscribe();
120112
}

projects/multidirectory-app/src/app/features/dns/add-zone-dialog/add-zone-dialog.component.ts

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,13 @@
11
import { Component, inject } from '@angular/core';
2-
import { DialogComponent } from '../../../components/modals/components/core/dialog/dialog.component';
2+
import { DialogComponent } from '@components/modals/components/core/dialog/dialog.component';
33
import { TranslocoModule } from '@jsverse/transloco';
44
import { FormsModule } from '@angular/forms';
55
import { DnsApiService } from '@services/dns-api.service';
66
import { DialogRef } from '@angular/cdk/dialog';
7-
import { DialogService } from '../../../components/modals/services/dialog.service';
7+
import { DialogService } from '@components/modals/services/dialog.service';
88
import { MultidirectoryUiKitModule } from 'multidirectory-ui-kit';
9-
import { IpListDialogComponent } from '@components/modals/components/dialogs/access-policy-ip-list/ip-list-dialog.component';
10-
import { IplistDialogData } from '@components/modals/interfaces/ip-list-dialog.interface';
11-
import { take } from 'rxjs';
12-
import { IpOption, IpRange } from '@core/access-policy/access-policy-ip-address';
13-
import { DnsAddZoneRequest, DnsZoneParam } from '@models/dhcp/areas/dhcp-add-areas-response';
9+
import { IpOption } from '@core/access-policy/access-policy-ip-address';
10+
import { DnsAddZoneRequest } from '@models/dhcp/areas/dhcp-add-areas-response';
1411
import { CommonModule } from '@angular/common';
1512
import { RequiredWithMessageDirective } from '@core/validators/required-with-message.directive';
1613

projects/multidirectory-app/src/app/features/dns/dns-forward-zones/dns-forward-zones.component.ts

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,14 @@
11
import { Component, inject, OnInit, signal } from '@angular/core';
2-
import { RedirectZonesRow } from '@features/dns/interfaces/redirect-zones.interface';
32
import { translate, TranslocoPipe } from '@jsverse/transloco';
4-
import { ButtonComponent, DatagridComponent, DropdownOption } from 'multidirectory-ui-kit';
3+
import { ButtonComponent, DatagridComponent } from 'multidirectory-ui-kit';
54
import { ToastrService } from 'ngx-toastr';
6-
import { DialogService } from '../../../components/modals/services/dialog.service';
5+
import { DialogService } from '@components/modals/services/dialog.service';
76
import { CommonModule } from '@angular/common';
87
import { DnsApiService } from '@services/dns-api.service';
98
import { DnsForwardGetData, DnsForwardZone } from '@models/api/dns/dns-forward-zone';
109
import { AddForwardZoneDialogData, AddForwardZoneDialogReturnData } from './add-forward-zone-dialog/add-forward-zone-dialog.interface';
1110
import { AddForwardZoneDialogComponent } from './add-forward-zone-dialog/add-forward-zone-dialog.component';
12-
import { DnsAddZoneRequest } from '@models/dhcp/areas/dhcp-add-areas-response';
13-
import { EMPTY, filter, switchMap, take } from 'rxjs';
11+
import { filter, switchMap, take } from 'rxjs';
1412
import { TableColumn } from 'ngx-datatable-gimefork';
1513

1614
@Component({
@@ -65,7 +63,7 @@ export class DnsForwardZonesComponent implements OnInit {
6563
return this.dnsApi.addForwardZone(result);
6664
}),
6765
)
68-
.subscribe((result) => {
66+
.subscribe(() => {
6967
this.loadData();
7068
});
7169
}
@@ -87,7 +85,7 @@ export class DnsForwardZonesComponent implements OnInit {
8785
return this.dnsApi.changeForwardZones(result);
8886
}),
8987
)
90-
.subscribe((result) => {
88+
.subscribe(() => {
9189
this.loadData();
9290
});
9391
}

projects/multidirectory-app/src/app/features/dns/dns-rule-dialog/dns-rule-dialog.component.html

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,15 +2,11 @@
22
<div class="app-modal-header">
33
<div>{{ 'dns-rule-dialog.modal-header' | transloco }}</div>
44
</div>
5-
<div class="flex align-center gap-1 pb-2">
5+
<div class="flex align-center gap-1 pb-2 mt-4">
66
<md-form #form>
77
<div class="form-grid">
88
<label>{{ 'dns-rule-dialog.name' | transloco }}</label>
9-
<md-textbox [(ngModel)]="dnsRule.record_name" [disabled]="sameAsZoneName || !!dialogData.isEdit" appRequired></md-textbox>
10-
<div></div>
11-
<md-checkbox [(ngModel)]="sameAsZoneName" [disabled]="!!dialogData.isEdit"
12-
>{{ 'dns-rule-dialog.same-as-zone-name' | transloco }}
13-
</md-checkbox>
9+
<md-textbox [(ngModel)]="dnsRule.record_name" [disabled]="!!dialogData.isEdit" appRequired></md-textbox>
1410
<label>{{ 'dns-rule-dialog.ip' | transloco }}</label>
1511
<div>
1612
@switch (recordDataType) {
@@ -29,8 +25,8 @@
2925
}
3026
</div>
3127
<label>{{ 'dns-rule-dialog.type' | transloco }}</label>
32-
<md-dropdown [(ngModel)]="dnsRule.record_type" [disabled]="!!dialogData.isEdit" [options]="DnsRuleTypes" appRequired></md-dropdown>
33-
<label>{{ 'dns-rule-dialog.ttl' | transloco }}</label>
28+
<md-dropdown [(ngModel)]="recordType" [disabled]="!!dialogData.isEdit" [options]="DnsRuleTypes" appRequired></md-dropdown
29+
><label>{{ 'dns-rule-dialog.ttl' | transloco }}</label>
3430
<md-textbox [(ngModel)]="dnsRule.ttl" appRequired></md-textbox>
3531
</div>
3632
</md-form>

projects/multidirectory-app/src/app/features/dns/dns-rule-dialog/dns-rule-dialog.component.ts

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -44,20 +44,6 @@ export class DnsRuleDialogComponent implements OnInit {
4444
private dialogService: DialogService = inject(DialogService);
4545
private dialogRef: DialogRef<DnsRuleDialogReturnData, DnsRuleDialogComponent> = inject(DialogRef);
4646

47-
private _sameAsZoneName = false;
48-
49-
get sameAsZoneName(): boolean {
50-
return this._sameAsZoneName;
51-
}
52-
53-
set sameAsZoneName(val: boolean) {
54-
this._sameAsZoneName = val;
55-
if (val) {
56-
this.dnsRule.record_name = '@';
57-
}
58-
this.cdr.detectChanges();
59-
}
60-
6147
get recordType() {
6248
return this.dnsRule.record_type;
6349
}

projects/multidirectory-app/src/app/features/dns/dns-rule-list-item/dns-rule-list-item.component.scss

Lines changed: 9 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,13 @@
11
.client-grid {
22
display: grid;
3-
grid-template-rows: 1fr 1fr;
4-
grid-template-columns: 32px 1fr 1fr 1fr 1fr 1fr 64px 32px;
3+
grid-template-rows: 1fr auto;
4+
grid-template-columns: 32px 1fr 1fr 1fr 64px 32px;
55
gap: 4px;
66
border: 1px solid var(--md-neutral-400);
77
border-radius: 8px;
88
background-color: var(--md-white);
99
min-height: 48px;
10+
padding: 12px 5px;
1011

1112
.cell-index {
1213
grid-column: 1;
@@ -36,8 +37,8 @@
3637
}
3738

3839
.cell-value {
39-
grid-row: 2 / 3;
40-
grid-column: 2 / 5;
40+
grid-row: 2;
41+
grid-column: 2/4;
4142
display: flex;
4243
flex-flow: column nowrap;
4344
justify-content: flex-start;
@@ -48,37 +49,28 @@
4849
}
4950

5051
.cell-type {
51-
grid-row: 1/3;
52-
grid-column: 6;
52+
grid-row: 2;
53+
grid-column: 4;
5354
overflow: hidden;
5455
text-overflow: ellipsis;
5556
align-self: center;
5657
}
5758

5859
.cell-ttl {
5960
grid-row: 2;
60-
grid-column: 3/5;
61+
grid-column: 5;
6162
overflow: hidden;
6263
text-overflow: ellipsis;
6364
}
6465

6566
.cell-control-buttons {
6667
display: flex;
6768
grid-row: 1/3;
68-
grid-column: 6 / 8;
69+
grid-column: 5;
6970
flex-flow: row nowrap;
7071
justify-content: flex-end;
7172
align-items: center;
7273
}
73-
74-
.cell-switches {
75-
display: flex;
76-
grid-row: 1/3;
77-
grid-column: 7;
78-
flex-flow: column nowrap;
79-
justify-content: center;
80-
align-items: flex-end;
81-
}
8274
}
8375

8476
.control-button-logo {

projects/multidirectory-app/src/app/features/dns/dns-zone-list-item/dns-zone-list-item.component.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ export class DnsZoneListItemComponent {
5757
this.dialog
5858
.open<any, DnsZoneDetailsDialogData, DnsZoneDetailsComponent>({
5959
component: DnsZoneDetailsComponent,
60-
dialogConfig: { data: { zoneName: record.name } },
60+
dialogConfig: { data: { zoneName: record.name }, width: '600px' },
6161
})
6262
.closed.pipe(take(1))
6363
.subscribe((x) => {});

projects/multidirectory-app/src/app/models/api/dns/dns-rule-type.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ export enum DnsRuleType {
55
AAAA = 'AAAA',
66
PTR = 'PTR',
77
CNAME = 'CNAME',
8-
ANAME = 'ANAME',
8+
ALIAS = 'ALIAS',
99
TXT = 'TXT',
1010
MX = 'MX',
1111
SRV = 'SRV',

projects/multidirectory-app/src/app/models/api/dns/dns-rule.ts

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,19 @@ import { DnsRuleType } from './dns-rule-type';
44
export class DnsRule {
55
record_name: string = '';
66
record_type: DnsRuleType = DnsRuleType['A'];
7-
record_value: string = '0 100 3268 md.localhost.';
7+
record_value: string = '';
88
ttl?: number = Constants.DnsTTL;
9-
content?: string = '0 100 3268 md.localhost.';
9+
10+
constructor(obj: Partial<DnsRule>) {
11+
Object.assign(this, obj);
12+
}
13+
}
14+
export class DnsRuleResponse {
15+
record_name: string = '';
16+
record_type: DnsRuleType = DnsRuleType['A'];
17+
record_value: string = '';
18+
ttl?: number = Constants.DnsTTL;
19+
content?: string = '';
1020

1121
constructor(obj: Partial<DnsRule>) {
1222
Object.assign(this, obj);

projects/multidirectory-app/src/app/models/api/setup/setup-request.ts

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -2,22 +2,22 @@ import { DnsSetupRequest } from '@models/api/dns/dns-setup-request';
22
import { DhcpSetupRequest } from '@models/api/dhcp/dhcp-setup-request';
33

44
export class SetupRequest {
5-
domain = '';
6-
username = '';
7-
user_principal_name = '';
8-
display_name = '';
9-
mail = '';
10-
password = '';
11-
repeatPassword = '';
5+
domain: string = '';
6+
username: string = '';
7+
user_principal_name: string = '';
8+
display_name: string = '';
9+
mail: string = '';
10+
password: string = '';
11+
repeatPassword: string = '';
1212

1313
setupKdc = true;
1414
setupDhcp = true;
1515
generateKdcPasswords = true;
16-
krbadmin_password = '';
17-
krbadmin_password_repeat = '';
16+
krbadmin_password: string = '';
17+
krbadmin_password_repeat: string = '';
1818

19-
stash_password = '';
20-
stash_password_repeat = '';
19+
stash_password: string = '';
20+
stash_password_repeat: string = '';
2121

2222
setupDns = true;
2323
setupDnsRequest = new DnsSetupRequest({});

0 commit comments

Comments
 (0)