Page MenuHomePhabricator

[web] reintroduce error message to thread settings modal
ClosedPublic

Authored by ginsu on Feb 12 2024, 2:13 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 18, 3:42 AM
Unknown Object (File)
Apr 8 2024, 11:19 AM
Unknown Object (File)
Apr 8 2024, 11:19 AM
Unknown Object (File)
Apr 8 2024, 11:19 AM
Unknown Object (File)
Apr 8 2024, 11:19 AM
Unknown Object (File)
Mar 12 2024, 8:09 PM
Unknown Object (File)
Mar 12 2024, 6:24 PM
Unknown Object (File)
Mar 12 2024, 5:30 PM
Subscribers

Details

Summary

This diff reintroduces the error message for the thread settings modal. Additionally, I deleted the SubmitSection component since this was no longer being used

Linear task: https://linear.app/comm/issue/ENG-5943/extendmodify-the-modal-props-api-to-follow-new-modal-designs

Depends on D11034

Test Plan

Please see the screenshots below

Screenshot 2024-02-12 at 5.15.16 AM.png (2×3 px, 884 KB)

Screenshot 2024-02-12 at 5.15.12 AM.png (2×3 px, 915 KB)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ginsu requested review of this revision.Feb 12 2024, 2:27 AM
This revision is now accepted and ready to land.Feb 12 2024, 1:06 PM