@sauceyred wrote in gorb/frontend#69 (comment):
Did you even look through the changes??
yeah, only took 2 minutes
well displayName seems wrong if you're talking about a server, or a role. I think of it has a member thing
i don't think "displayName" is the correct term here, assuming this is a generic confirmation screen, either rename the prop to "target" or something, or rename the modal
splicing like this seems really messy, there should really be a better way of doing this
don't believe so, i'd just ignore the frontend PR for now, i'll just move the files manually when i work on it again
@sauceyred wrote in gorb/frontend#81 (comment):
In the future, please don't include all changes made in response to a review as a singular commit.
yeahh,…