Page MenuHomePhabricator

[web] fix prop drilling within relationships modals
ClosedPublic

Authored by ginsu on Jan 19 2024, 11:34 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 14, 1:32 AM
Unknown Object (File)
Apr 13 2024, 10:29 PM
Unknown Object (File)
Apr 8 2024, 4:32 PM
Unknown Object (File)
Apr 8 2024, 4:32 PM
Unknown Object (File)
Apr 8 2024, 4:32 PM
Unknown Object (File)
Apr 8 2024, 4:32 PM
Unknown Object (File)
Apr 8 2024, 4:32 PM
Unknown Object (File)
Mar 6 2024, 3:10 PM
Subscribers

Details

Summary

PLEASE NOTE THAT THIS DIFF AND SUBSEQUENT DIFFS IN THIS STACK WILL NOT BE LANDED UNTIL MORE OF THE REDESIGN IS READY SINCE THIS WILL CAUSE REGRESSIONS IN PROD

As I was working with the relationship modals/panels I found that we were drilling this onClose prop method that pretty much was just popModal. We don't need to drill this down everywhere and instead we can just call popModal using useModalContext hook where we need it.

Depends on D10742

Test Plan

flow + confirmed that the modal on close behavoir was popping as expected

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Jan 19 2024, 4:18 PM
This revision was landed with ongoing or failed builds.Feb 15 2024, 1:18 AM
This revision was automatically updated to reflect the committed changes.