Sharing modal update

this shouldn't be checked in