"Add email" button showing incorrectly on a private domain's workspace

[kevinksullivan]

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!

---

Action Performed:

1. Create workspace (with [email protected])
2. Go to Workspace menu

Expected Result:

The button should say Get Started

Actual Result:

The button incorrectly says Add email, and is asking to add a private email address, on STAGING only.

On prod (shown correctly)

On staging (shown incorrectly

Workaround:

None. I cannot enable the card.

Version Number: v1.0.99-0

View all open jobs on GitHub

[MelvinBot]

Triggered auto assignment to @mountiny (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[OSBotify]

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

1. Identify the pull request that introduced this issue and revert it.
2. Find someone who can quickly fix the issue.
3. Fix the issue yourself.

[luacmartins]

This is probably related to this PR #5037. We probably don't have the domain in the common domain list or bedrock cache. That means we are actually going to Clearbit for certainty and meanwhile we treat the domain as public.

Do we require certainty that the domain is private in this case?

[marcaaron]

Just to follow up here. I'm still a bit unsure why the PRs we are reverting will fix anything and did not really get very far in reproducing this issue beyond the realization that email validation for non obviously public domains seems broken on dev (I used the clitools account validator and it was broken). So I'd guess (but did not confirm) that some users may have experienced issues validating accounts during the short time this code was live.

The error I saw is here.

Once I locally reverted the lines here I was able to validate a non-public domain just fine without throwing any errors and logged in OK in New Expensify on web.

https://github.com/Expensify/Web-Expensify/blob/576c72e2325caf9bfb6f8a318ac9d82add88fc9e/lib/UserAPI.php#L1818-L1823

After creating and navigating to a workspace I still did not see an Add Email button like Kevin shared.

So TL;DR revert seems like a good call. But whether it fixed this issue I'm not too sure.

@luacmartins If you are up for it at some point (tomorrow maybe since it's late 😅 ) it would be good to provide a summary on this issue to explain why @kevinksullivan experienced this if it's obvious?

[luacmartins]

Following up on this issue. I didn't manage to reproduce the issue @kevinksullivan had, but it seems that reverting the PRs resolved the issue, so I think we can close this issue 🎉

The Clearbit exception being thrown here, as pointed out by @marcaaron, seems to be the problem in this case. However, I'm still unsure why new domains did not experience that behavior before these PRs, because AFAIK we triggered that same Clearbit exception while calling isFromPublicDomain. I'll try to investigate this more as part of this comment.