fix(a11y): alert files reloaded on mgt-file-list

Mnickii commented 1 month ago

Closes #3097

PR Type

Bugfix

Description of the changes

PR checklist

[x] Project builds (yarn build) and changes have been tested in at least two supported browsers (Edge + non-Chromium based browser)
[ ] All public APIs (classes, methods, etc) have been documented following the jsdoc syntax
[x] Stories have been added and existing stories have been tested
[ ] Added appropriate documentation. Docs PR:
[ ] License header has been added to all new source files (yarn setLicense)
[x] Contains NO breaking changes

Other information

microsoft-github-policy-service[bot] commented 1 month ago

Thank you for creating a Pull Request @Mnickii.

This is a checklist for the PR reviewer(s) to complete before approving and merging this PR:

[ ] I have verified a documentation PR has been linked and is approved (or not applicable)
[ ] I have ran this PR locally and have tested the fix/feature
[ ] I have verified that stories have been added to storybook (or not applicable)
[ ] I have tested existing stories in storybook to verify no regression has occured
[ ] I have tested the solution in at least two browsers (Edge + 1 non-Chromium based browser)

github-actions[bot] commented 1 month ago

📖 The updated storybook is available here

github-actions[bot] commented 1 month ago

📖 The updated storybook is available here

github-actions[bot] commented 1 month ago

📖 The updated storybook is available here

github-actions[bot] commented 1 month ago

📖 The updated storybook is available here

github-actions[bot] commented 1 month ago

Package	Line Rate	Branch Rate	Health
mgt-components.src.components	86%	75%	✔
mgt-components.src.components.mgt-contact	68%	83%	➖
mgt-components.src.components.mgt-file	62%	100%	➖
mgt-components.src.components.mgt-file-list	46%	100%	❌
mgt-components.src.components.mgt-file-list.mgt-file-upload	49%	86%	❌
mgt-components.src.components.mgt-get	22%	100%	❌
mgt-components.src.components.mgt-messages	68%	100%	➖
mgt-components.src.components.mgt-organization	47%	100%	❌
mgt-components.src.components.mgt-person	82%	100%	✔
mgt-components.src.components.mgt-person-card	77%	49%	✔
mgt-components.src.components.mgt-picker	80%	100%	✔
mgt-components.src.components.mgt-profile	40%	100%	❌
mgt-components.src.components.mgt-tasks-base	87%	100%	✔
mgt-components.src.components.mgt-theme-toggle	100%	100%	✔
mgt-components.src.components.mgt-todo	79%	100%	✔
mgt-components.src.components.sub-components.mgt-flyout	78%	53%	✔
mgt-components.src.components.sub-components.mgt-spinner	100%	100%	✔
mgt-components.src.graph	38%	78%	❌
mgt-components.src.styles	92%	80%	✔
mgt-components.src.utils	79%	27%	✔
mgt-element.dist.es6.components.src.components	72%	73%	➖
mgt-element.dist.es6.mock.src.mock	91%	73%	✔
mgt-element.dist.es6.providers.src.providers	85%	76%	✔
mgt-element.dist.es6.src	91%	80%	✔
mgt-element.dist.es6.utils.src.utils	68%	71%	➖
mgt-element.src	93%	40%	✔
mgt-element.src.components	78%	100%	✔
mgt-element.src.mock	81%	56%	✔
mgt-element.src.providers	84%	100%	✔
mgt-element.src.utils	72%	91%	➖
Summary	67% (14551 / 21559)	74% (519 / 699)	➖

Mnickii commented 1 month ago

@Raisul123 @vagpt this is ready for review

vagpt commented 1 month ago

@Raisul123 @vagpt this is ready for review

This issue is working fine on the URL below, hence we are good to close this issue.

URL - https://mgt.dev/next/pr/3191/?path=/story/components-mgt-file-list-properties--clear-cache-and-reload

Attachment:

https://github.com/microsoftgraph/microsoft-graph-toolkit/assets/95913748/a8187e97-2581-4cd6-9a19-cabe3f6a954a

microsoftgraph / microsoft-graph-toolkit