Skip to content

Comments

Validation + UI update when trying to create Quality Inspection Results records with empty fields.#6805

Open
6CRIPT wants to merge 2 commits intomainfrom
bugs/622623-AddRedAsterisToTheDescriptionField
Open

Validation + UI update when trying to create Quality Inspection Results records with empty fields.#6805
6CRIPT wants to merge 2 commits intomainfrom
bugs/622623-AddRedAsterisToTheDescriptionField

Conversation

@6CRIPT
Copy link

@6CRIPT 6CRIPT commented Feb 24, 2026

The Code and Description fields now display a red asterisk to indicate that they are mandatory. An error is shown if the window is closed or if a new record is inserted or an existing one modified with empty fields.

Fixes AB#622623

…at thay can not be empty and the window throws an error if you try to close it/insert one new record/modify an existing record with empty fields.
@6CRIPT 6CRIPT requested a review from a team as a code owner February 24, 2026 07:49
@github-actions github-actions bot added the AL: Apps (W1) Add-on apps for W1 label Feb 24, 2026
@github-actions
Copy link

Issue #622623 is not valid. Please make sure you link an issue that exists, is open and is approved.

@github-actions github-actions bot added this to the Version 29.0 milestone Feb 24, 2026
@6CRIPT
Copy link
Author

6CRIPT commented Feb 24, 2026

@microsoft-github-policy-service agree company="Microsoft"

1 similar comment
@6CRIPT
Copy link
Author

6CRIPT commented Feb 24, 2026

@microsoft-github-policy-service agree company="Microsoft"

if Rec.Code = DefaultResultInProgressCodeLbl then
Rec."Finish Allowed" := Rec."Finish Allowed"::"Do Not Allow Finish";

TestField("Description");
Copy link
Contributor

@AndreiPanko AndreiPanko Feb 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please don't

we already have:
field(2; "Description"; Text[100])
{
Caption = 'Description';
NotBlank = true;
ToolTip = 'Specifies a friendly description for the result.';
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AL: Apps (W1) Add-on apps for W1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants