CL Fisher
CL Fisher
We are still working on getting this reviewed.
Hey @edwinthinks -- IIRTC Looks like you've still got tests that call OrganizationController.invite_user, which you've deleted?
With this branch, I tried to invite myself as a bank user on my local So I think there's still real work to be done on this.
@dorner Given your level of busy, can I pass this off to @awwaiid for review instead of you? It looks like you were close to oking it before.
Finally! I'm not going to try to get it in for this weekend -- it's already pretty meaty -- but I will kick the tires.
@dorner From tonight's stakeholder meeting -- can we please limit access to this to org_admins ? Thanks.
> The bug you found was because you were inviting a user that already existed in the system - and that user doesn't have an `invitation_sent_at`. Pretty sure this is...
@awwaiid Can you take a look -- mostly as a formality -- @dorner took this over to get it to the finish line.
The status on this is waiting for @awwaiid to take a look at it then.
Hey @jadekstewart3 -- a question rather than an actual review.... You are proposing a new partial -- Is the partial that this is currently in one of the ones that...