From c787229a3f69335bca312e84b46faad7d726589e Mon Sep 17 00:00:00 2001 From: JaySoni1 Date: Fri, 20 Feb 2026 02:51:51 +0530 Subject: [PATCH] WEB-760 Validation and error message improvements for Add Column dialog --- .../column-dialog/column-dialog.component.html | 13 +++++++++++++ .../column-dialog/column-dialog.component.ts | 5 ++++- 2 files changed, 17 insertions(+), 1 deletion(-) diff --git a/src/app/system/manage-data-tables/column-dialog/column-dialog.component.html b/src/app/system/manage-data-tables/column-dialog/column-dialog.component.html index d916887e7..b70c53262 100644 --- a/src/app/system/manage-data-tables/column-dialog/column-dialog.component.html +++ b/src/app/system/manage-data-tables/column-dialog/column-dialog.component.html @@ -49,9 +49,16 @@ matInput required type="number" + min="1" formControlName="length" placeholder="{{ 'labels.inputs.Column Length' | translate }}" /> + @if (columnForm.controls.length.hasError('required')) { + + {{ 'labels.inputs.Column Length' | translate }} {{ 'labels.commons.is' | translate }} + {{ 'labels.commons.required' | translate }} + + } } @@ -65,6 +72,12 @@ } + @if (columnForm.controls.code.hasError('required')) { + + {{ 'labels.inputs.Column Code' | translate }} {{ 'labels.commons.is' | translate }} + {{ 'labels.commons.required' | translate }} + + } } diff --git a/src/app/system/manage-data-tables/column-dialog/column-dialog.component.ts b/src/app/system/manage-data-tables/column-dialog/column-dialog.component.ts index 3693b8a77..da8c5a6d4 100644 --- a/src/app/system/manage-data-tables/column-dialog/column-dialog.component.ts +++ b/src/app/system/manage-data-tables/column-dialog/column-dialog.component.ts @@ -76,7 +76,10 @@ export class ColumnDialogComponent implements OnInit { value: this.data ? +this.data.columnLength : '', disabled: this.getColumnType(this.data.columnDisplayType) !== 'String' || this.data.type === 'existing' }, - Validators.required + [ + Validators.required, + Validators.min(1) + ] ], mandatory: [{ value: this.data.isColumnNullable, disabled: this.data.type === 'existing' }], unique: [