CL Fisher
CL Fisher
@dorner -- sorry, I neglected to indicate that I think this is ready for the first pass of a technical review -- the two things I've noted are quite minor...
Ok @coalest -- We still need those two minor fixes, but once those are in we'll be good to go.
Hey @coalest -- are you able to finish this off (or should we get someone else to handle those last tweaks)? Thanks!
@dorner -- Could you take a quick look at my commits and make sure they're up to snuff before we merge this? Thanks.
I agree with Brock -- I really don't think we're going back at this point. I'm not quite ready to rip out the old stuff entirely , but we should...
This looks good to me functionally. I have added making the new users add a role instead of the organization to the project's copious inbox.
@dorner Can you take a final look at this?
I think it'll be simplest to hold off on merging this one into main until we get the other fix around this in.
@Naraveni Sorry this seems to have fallen through the cracks. I addressed some conflicts that had arisen with the current main branch, and took a look at it for any...
@dorner -- do you want to take a technical pass on this? My nitpick is small enough that we could just edit the file on behalf of @jahuang415 , methinks.