Skip to content

Check for the validation of mandatory fields after the fix #1813

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
cs-raj wants to merge 2 commits into from
Closed

Check for the validation of mandatory fields after the fix #1813

cs-raj wants to merge 2 commits into from

Conversation

@cs-raj
Copy link
Contributor

@cs-raj cs-raj commented Mar 17, 2025

No description provided.

@cs-raj cs-raj requested a review from a team as a code owner March 17, 2025 09:51
sunil-lakshman
sunil-lakshman approved these changes Mar 17, 2025
View reviewed changes
Copy link
Contributor

@sunil-lakshman sunil-lakshman left a comment

Choose a reason for hiding this comment

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

LGTM

@cs-raj
Copy link
Contributor Author

cs-raj commented Mar 17, 2025

Hi Team,
Closing this PR since if a field is mandatory and it does not contain any data it will be removed already so the check for mandatory field after the field fixing will add no benefit
cc: @netrajpatel @shafeeqd959 @aman1908

@cs-raj cs-raj closed this Mar 17, 2025
@cs-raj cs-raj deleted the fix/8x8 branch March 17, 2025 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sunil-lakshman sunil-lakshman sunil-lakshman approved these changes

@shafeeqd959 shafeeqd959 Awaiting requested review from shafeeqd959

@aman19K aman19K Awaiting requested review from aman19K

@harshithad0703 harshithad0703 Awaiting requested review from harshithad0703

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@cs-raj @sunil-lakshman