[MHV-55186] Update all V1 components to V3 web components on SM MHV #41811
pull-request.yml
on: pull_request
Fetch Unit Test Stability Allow List
1m 1s
ESLint Disable Check
1m 1s
Icon Check
57s
Sentry Check
1m 2s
Linting (Files Changed)
1m 8s
Check Cross App Imports
5m 36s
Annotations
10 errors and 25 warnings
Linting (Files Changed):
src/applications/mhv/secure-messaging/tests/e2e/pages/FolderManagementPage.js#L133
src/applications/mhv/secure-messaging/tests/e2e/pages/FolderManagementPage.js:133:38:Replace `mockParentMessageDetails.data.attributes.messageId` with `⏎········mockParentMessageDetails.data.attributes.messageId⏎······`
|
Linting (Files Changed):
src/applications/mhv/secure-messaging/tests/e2e/pages/FolderManagementPage.js#L202
src/applications/mhv/secure-messaging/tests/e2e/pages/FolderManagementPage.js:202:42:Replace `mockCustomResponse.data.attributes.threadId` with `⏎········mockCustomResponse.data.attributes.threadId⏎······`
|
Linting (Files Changed):
src/applications/mhv/secure-messaging/tests/e2e/pages/FolderManagementPage.js#L223
src/applications/mhv/secure-messaging/tests/e2e/pages/FolderManagementPage.js:223:41:Replace `mockCustomResponse.data.attributes.threadId` with `⏎········mockCustomResponse.data.attributes.threadId⏎······`
|
Linting (Files Changed):
src/applications/mhv/secure-messaging/tests/e2e/pages/FolderManagementPage.js#L239
src/applications/mhv/secure-messaging/tests/e2e/pages/FolderManagementPage.js:239:41:Replace `mockCustomResponse.data.attributes.threadId` with `⏎········mockCustomResponse.data.attributes.threadId⏎······`
|
Linting (Files Changed):
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientComposePage.js#L205
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientComposePage.js:205:49:Replace `draftMessage.data.attributes.messageId` with `⏎········draftMessage.data.attributes.messageId⏎······`
|
Linting (Files Changed):
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientComposePage.js#L362
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientComposePage.js:362:38:Replace `mockMessageResponse.data.attributes.messageId` with `⏎········mockMessageResponse.data.attributes.messageId⏎······`
|
Linting (Files Changed):
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientInboxPage.js#L145
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientInboxPage.js:145:54:Replace `inputMockMessage.attributes.messageId` with `⏎········inputMockMessage.attributes.messageId⏎······`
|
Linting (Files Changed):
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientInboxPage.js#L167
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientInboxPage.js:167:35:Replace `mockMessages.data[0].attributes.messageId` with `⏎········mockMessages.data[0].attributes.messageId⏎······`
|
Linting (Files Changed):
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientInboxPage.js#L172
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientInboxPage.js:172:35:Replace `this.singleThread.data[0].attributes.messageId` with `⏎········this.singleThread.data[0].attributes.messageId⏎······`
|
Linting (Files Changed):
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientInboxPage.js#L179
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientInboxPage.js:179:37:Replace `this.singleThread.data[1].attributes.messageId` with `⏎··········this.singleThread.data[1].attributes.messageId⏎········`
|
Icon Check
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: department-of-veterans-affairs/action-inject-ssm-secrets@latest. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
ESLint Disable Check
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: department-of-veterans-affairs/action-inject-ssm-secrets@latest. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
Fetch Unit Test Stability Allow List
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: department-of-veterans-affairs/action-inject-ssm-secrets@latest, actions/checkout@v3, actions/setup-node@v3, actions/upload-artifact@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
Sentry Check
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: department-of-veterans-affairs/action-inject-ssm-secrets@latest. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
Linting (Files Changed)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: trilom/file-changes-action@v1.2.4. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
Linting (Files Changed)
The following actions uses node12 which is deprecated and will be forced to run on node16: trilom/file-changes-action@v1.2.4. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
Linting (Files Changed):
src/applications/mhv/secure-messaging/components/AttachmentsList.jsx#L3
src/applications/mhv/secure-messaging/components/AttachmentsList.jsx:3:1:Importing from platform via platform/monitoring is deprecated. Import from @department-of-veterans-affairs/platform-monitoring instead or check babel.config.json for an alias to the import.
|
Linting (Files Changed):
src/applications/mhv/secure-messaging/components/AttachmentsList.jsx#L69
src/applications/mhv/secure-messaging/components/AttachmentsList.jsx:69:5:React Hook useEffect has a missing dependency: 'setAttachFileSuccess'. Either include it or remove the dependency array.
|
Linting (Files Changed):
src/applications/mhv/secure-messaging/components/AttachmentsList.jsx#L175
src/applications/mhv/secure-messaging/components/AttachmentsList.jsx:175:19:The <va-button> Web Component should be used instead of the button HTML element.
|
Linting (Files Changed):
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx#L165
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx:165:5:React Hook useEffect has a missing dependency: 'defaultRecipientsList'. Either include it or remove the dependency array.
|
Linting (Files Changed):
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx#L200
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx:200: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#L231
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx:231:5:React Hook useEffect has missing dependencies: 'attachments', 'category', 'currentFolder?.folderId', 'dispatch', 'draft?.messageId', 'history', 'messageBody', 'selectedRecipient', and 'subject'. Either include them or remove the dependency array.
|
Linting (Files Changed):
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx#L249
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx:249:5:React Hook useEffect has a missing dependency: 'lastFocusableElement'. Either include it or remove the dependency array.
|
Linting (Files Changed):
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx#L445
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx:445:5:React Hook useEffect has missing dependencies: 'draft?.body' and 'savedDraft'. Either include them or remove the dependency array.
|
Linting (Files Changed):
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx#L524
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx:524: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#L544
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx:544:5:React Hook useEffect has a missing dependency: 'noTimeout'. Either include it or remove the dependency array.
|
Linting (Files Changed)
Received event from pull_request, but also received a before(fdbb3a6db966777a1bd6273a1faf3c27afa8f268) or after(7320089267d38a09262ead627c7e0f3ae6f9d559) value.
I am assuming you want to use a Push event but forgot something, so I'm giving you a message.
|
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)
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)
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)
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/
|
Check Cross App Imports
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/download-artifact@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
Artifacts
Produced during runtime
Name | Size | |
---|---|---|
unit-test-allow-list
|
552 KB |
|