Skip to content

[MHV-50504] Update navigating away user flows for compose and reply d… #57643

[MHV-50504] Update navigating away user flows for compose and reply d…

[MHV-50504] Update navigating away user flows for compose and reply d… #57643

Triggered via pull request October 4, 2024 00:06
Status Success
Total duration 7m 33s
Artifacts 1

pull-request.yml

on: pull_request
Fetch Unit Test Stability Allow List
54s
Fetch Unit Test Stability Allow List
ESLint Disable Check
0s
ESLint Disable Check
Icon Check
0s
Icon Check
Sentry Check
0s
Sentry Check
Linting (Files Changed)
1m 41s
Linting (Files Changed)
Check Cross App Imports
6m 22s
Check Cross App Imports
Fit to window
Zoom out
Zoom in

Annotations

12 warnings
Linting (Files Changed)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Linting (Files Changed)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Linting (Files Changed): src/applications/mhv-secure-messaging/components/ComposeForm/ComposeForm.jsx#L49
src/applications/mhv-secure-messaging/components/ComposeForm/ComposeForm.jsx:49:11:'pageTitle' is missing in props validation
Linting (Files Changed): src/applications/mhv-secure-messaging/components/ComposeForm/ComposeForm.jsx#L49
src/applications/mhv-secure-messaging/components/ComposeForm/ComposeForm.jsx:49:22:'headerRef' is missing in props validation
Linting (Files Changed): src/applications/mhv-secure-messaging/components/ComposeForm/ComposeForm.jsx#L241
src/applications/mhv-secure-messaging/components/ComposeForm/ComposeForm.jsx:241:6:React Hook useEffect has missing dependencies: 'draft' and 'recipients'. Either include them or remove the dependency array.
Linting (Files Changed): src/applications/mhv-secure-messaging/components/ComposeForm/ComposeForm.jsx#L281
src/applications/mhv-secure-messaging/components/ComposeForm/ComposeForm.jsx:281:5:React Hook useEffect has missing dependencies: 'attachments', 'category', 'currentFolder?.folderId', 'dispatch', 'draft?.messageId', 'electronicSignature', 'history', 'messageBody', 'selectedRecipient', and 'subject'. Either include them or remove the dependency array. Outer scope values like 'scrollToTop' aren't valid dependencies because mutating them doesn't re-render the component.
Linting (Files Changed): src/applications/mhv-secure-messaging/components/ComposeForm/ComposeForm.jsx#L629
src/applications/mhv-secure-messaging/components/ComposeForm/ComposeForm.jsx:629:5:React Hook useEffect has a missing dependency: 'setUnsavedNavigationError'. Either include it or remove the dependency array.
Linting (Files Changed): src/applications/mhv-secure-messaging/components/ComposeForm/ComposeForm.jsx#L648
src/applications/mhv-secure-messaging/components/ComposeForm/ComposeForm.jsx:648:5:React Hook useCallback has unnecessary dependencies: 'setCheckboxMarked' and 'setElectronicSignature'. Either exclude them or remove the dependency array.
Linting (Files Changed): src/applications/mhv-secure-messaging/components/ComposeForm/ComposeForm.jsx#L708
src/applications/mhv-secure-messaging/components/ComposeForm/ComposeForm.jsx:708:5:React Hook useCallback has missing dependencies: 'noTimeout' and 'signOutMessage'. Either include them or remove the dependency array.
Linting (Files Changed): src/applications/mhv-secure-messaging/components/ComposeForm/ComposeForm.jsx#L728
src/applications/mhv-secure-messaging/components/ComposeForm/ComposeForm.jsx:728:5:React Hook useEffect has a missing dependency: 'noTimeout'. Either include it or remove the dependency array.
Linting (Files Changed): src/applications/mhv-secure-messaging/components/ComposeForm/ReplyDraftItem.jsx#L137
src/applications/mhv-secure-messaging/components/ComposeForm/ReplyDraftItem.jsx:137:5:React Hook useCallback has missing dependencies: 'noTimeout' and 'signOutMessage'. Either include them or remove the dependency array.
Linting (Files Changed): src/applications/mhv-secure-messaging/components/ComposeForm/ReplyDraftItem.jsx#L149
src/applications/mhv-secure-messaging/components/ComposeForm/ReplyDraftItem.jsx:149:5:React Hook useEffect has a missing dependency: 'noTimeout'. Either include it or remove the dependency array.

Artifacts

Produced during runtime
Name Size
unit-test-allow-list
34.6 KB