[web] Turn `InvalidUploadModal` into a functional component
ClosedPublic
Actions

Authored by atul on Jun 2 2022, 10:15 PM.

Details

Reviewers

tomek
• jacek

Commits

rCOMM9801af9b88c2: [web] Turn `InvalidUploadModal` into a functional component

Summary

Was looking at this as part of the work to create a common Alert/Prompt component and noticed it might as well be a functional component... so made that change.

Test Plan

Dragged a random non-image file into the chat area and the InvalidUploadModal component was displayed as expected:

Screen Shot 2022-06-03 at 1.13.27 AM.png (456×726 px, 33 KB)

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Jun 2 2022, 10:15 PM

Herald added subscribers: • abosh, • adrian, ashoat. · View Herald TranscriptJun 2 2022, 10:15 PM

atul edited the test plan for this revision. (Show Details)Jun 2 2022, 10:15 PM

Harbormaster completed remote builds in B9611: Diff 13322.Jun 2 2022, 10:22 PM

atul requested review of this revision.Jun 2 2022, 10:22 PM

atul added a child revision: D4207: [web] Introduce minimally viable `Alert` component.Jun 2 2022, 11:18 PM

tomek accepted this revision.Jun 3 2022, 4:38 AM

tomek added inline comments.

web/modals/chat/invalid-upload.react.js
11–17 ↗	(On Diff #13322)

This revision is now accepted and ready to land.Jun 3 2022, 4:38 AM

cherrypick before landing

Harbormaster completed remote builds in B9620: Diff 13333.Jun 4 2022, 4:50 PM

address feedback

Harbormaster completed remote builds in B9621: Diff 13334.Jun 4 2022, 5:11 PM

Closed by commit rCOMM9801af9b88c2: [web] Turn `InvalidUploadModal` into a functional component (authored by atul). · Explain WhyJun 4 2022, 6:01 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM9801af9b88c2: [web] Turn `InvalidUploadModal` into a functional component.

Revision Contents
Changeset List

Path

Size

web/

modals/

chat/

invalid-upload.react.js

46 lines

Diff 13335

View Options

web/modals/chat/invalid-upload.react.js

[web] Turn `InvalidUploadModal` into a functional componentClosedPublicActions