[PM-22405] - Remove invalid users from invites

[jrmccannon]

🎟️ Tracking

PM-22405

📔 Objective

As a user goes to bulk resend invites, it attempts to match the Organization Users against the existing users. With those SSO JIT OrgUsers, the null email will cause the database call to fail because it can't map the NULL value to an Email.

In order to unblock current users who are experiencing this issue, we're logging the users with an invalid state and then not attempting to send an invite email to those users. This is because the token generated requires a matching email address to be present on the invited organization user during validation. Since we can't send the email, we'll just drop them.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.02%. Comparing base (cded079) to head (fe10237).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #8151   +/-   ##
=======================================
  Coverage   63.02%   63.02%           
=======================================
  Files        2316     2316           
  Lines      100453   100475   +22     
  Branches     9037     9038    +1     
=======================================
+ Hits        63306    63329   +23     
+ Misses      34962    34961    -1     
  Partials     2185     2185           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[eliykat]

the token generated requires a matching email address to be present on the invited organization user during validation

In other words: even if we send the email, the user can't accept the link, because the accept flow can't find them in this state?

What about your original "self-healing" logic - wouldn't that address this?