cms icon indicating copy to clipboard operation
cms copied to clipboard

hit checkUserEmailForPurchase in refreshDB only in prod

Open VK-RED opened this issue 1 year ago • 2 comments

PR Fixes:

    1. Earlier, we were hitting checkUserEmailForPurchase in refresDb in both local and prod
    1. Since in local we don't have APPX Creds, we now only hit it in production.

Resolves #402

Screencast (Before fix)

Screencast from 14-04-24 11:47:05 AM IST.webm

Screencast (After fix)

Screencast from 14-04-24 11:48:13 AM IST.webm

Checklist before requesting a review

  • I have performed a self-review of my code
  • I assure there is no similar/duplicate pull request regarding same issue

VK-RED avatar Apr 14 '24 06:04 VK-RED

refreshDB should just early return for local development No need to drill this down to the checkUserEmailForPurchase function checkUserEmailForPurchase should only be called in prod

hkirat avatar Apr 14 '24 13:04 hkirat

Yup !! Agreed !!

VK-RED avatar Apr 14 '24 13:04 VK-RED