Skip to content
This repository was archived by the owner on Aug 7, 2025. It is now read-only.

974372: SyncfusionExamples (FileFormatLibraries) - Github Dependabot security alerts #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Johnson-SF3639
Copy link
Collaborator

Bug description

Need to improve the incorrect usage of html attributes in markup of DataGrid
task : https://dev.azure.com/EssentialStudio/Ej2-Web/_workitems/edit/959908

Root cause

The problem occurs due to

Reason for not identifying earlier

Find how it was missed in our earlier testing and development by analyzing the below checklist. This will help prevent similar mistakes in the future.

  • Guidelines/documents are not followed

  • Common guidelines / Core team guideline

  • Specification document

  • Requirement document

  • Guidelines/documents are not given

  • Common guidelines / Core team guideline

  • Specification document

  • Requirement document

Reason:

Guidelines/documents are not given - Requirement document

Action taken:

Added the manual testing sample to ensure this issue in future. You can find the testbed sample in the below location,

Related areas:

lazyloading, Grouping and Show/Hide

Is it a breaking issue?

No

Solution description

The issue was resolved by

Output screenshots

Before fix

After fix

Areas affected and ensured

show-hide.ts file show and hide method

Ensured the following cases:

  • Verified that

Additional checklist

This may vary for different teams or products. Check with your scrum masters.

  • Did you run the automation against your fix? - Yes

  • Is there any API name change? - No

  • Is there any existing behavior change of other features due to this code change? - No

  • Does your new code introduce new warnings or binding errors? - No

  • Does your code pass all FxCop and StyleCop rules? a- No

  • Did you record this case in the unit test or UI test? - Yes

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant