Closed gor181 closed 8 years ago
Remove the confirm logic from UpdateShoutModal and move it to a new component included by ShoutPage Create a GenericModal which takes the optional number of buttons and their callbacks
confirm
UpdateShoutModal
ShoutPage
Hi @gor181 looks good to me, thanks 👍 Let me just suggest some small refactoring to keep consistency with the existing code base :)
Remove the
confirm
logic fromUpdateShoutModal
and move it to a new component included byShoutPage
Create a GenericModal which takes the optional number of buttons and their callbacks