App icon indicating copy to clipboard operation
App copied to clipboard

[$250] User can't download the production app from playstore

Open mvtglobally opened this issue 2 years ago • 34 comments

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. Go to playstore
  2. Download new expensify
  3. Open the app and login
  4. Check if it has a stg badge at top

Expected Result:

The app should not have the stg badge and stg features

Actual Result:

The app is stg

Workaround:

unknown

Platform:

Where is this issue occurring?

  • Android

Version Number: Reproducible in staging?: need repro Reproducible in production?: need repro Email or phone of affected tester (no customers): Logs: https://stackoverflow.com/c/expensify/questions/4856 Notes/Photos/Videos: Any additional supporting documentation Expensify/Expensify Issue URL: Issue reported by: @Puneet-here Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1662562500746549?thread_ts=1660710162.573049&cid=C01GTK53T8Q

View all open jobs on GitHub

mvtglobally avatar Sep 20 '22 05:09 mvtglobally

@mvtglobally are you able to reproduce?

mallenexpensify avatar Oct 12 '22 00:10 mallenexpensify

@mvtglobally Huh... This is 4 days overdue. Who can take care of this?

melvin-bot[bot] avatar Oct 17 '22 07:10 melvin-bot[bot]

@mvtglobally can you see if you're able to reproduce?

mallenexpensify avatar Oct 17 '22 17:10 mallenexpensify

@mvtglobally 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

melvin-bot[bot] avatar Oct 19 '22 06:10 melvin-bot[bot]

@mallenexpensify @mallenexpensify I was checking with the team on the same. There is a tricky way to download PROD build on Android with the same device as you have staging. Currently we all see STG Since all our testers are already getting BETA (STG) builds, not sure this result is accurate. Is there any chance you can ask someone in Expensify internally who DOES NOT have staging build to download PROD build?

https://user-images.githubusercontent.com/43995119/196693331-402beb99-5498-4497-b2a6-ca15d1eaeb11.mp4

mvtglobally avatar Oct 19 '22 12:10 mvtglobally

Asking internally https://expensify.slack.com/archives/C01SKUP7QR0/p1666291488048139

mallenexpensify avatar Oct 20 '22 18:10 mallenexpensify

Gonna close cuz @alex-mechler was able to help test and it looks like its not repro image

image

mallenexpensify avatar Oct 20 '22 18:10 mallenexpensify

Be sure to fill out the Contact List!

melvin-bot[bot] avatar Oct 20 '22 18:10 melvin-bot[bot]

@mallenexpensify, in the screenshot I can see the stg badge at the login page.

Puneet-here avatar Oct 20 '22 19:10 Puneet-here

ah shit... you're right @Puneet-here , great 👀 .
Reopening and assigning an engineer to take a look

mallenexpensify avatar Oct 20 '22 19:10 mallenexpensify

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

melvin-bot[bot] avatar Oct 20 '22 19:10 melvin-bot[bot]

Oh, I think this is a dupe of https://github.com/Expensify/Expensify/issues/237206 and it is probably internal

iwiznia avatar Oct 21 '22 11:10 iwiznia

Oh wait, maybe not. Forget that.

iwiznia avatar Oct 21 '22 11:10 iwiznia

Ummm, isn't play store downloading the staging app because you are in the beta???

iwiznia avatar Oct 21 '22 11:10 iwiznia

Ummm, isn't play store downloading the staging app because you are in the beta???

No, I don't think this is the case because external contributors aren't in the beta and I have tried some different accounts that I've never used at expensify but I still couldn't download the production app.

Puneet-here avatar Oct 21 '22 11:10 Puneet-here

Ah ok ok, thanks for the context

iwiznia avatar Oct 21 '22 11:10 iwiznia

Current assignee @mallenexpensify is eligible for the External assigner, not assigning anyone new.

melvin-bot[bot] avatar Oct 21 '22 11:10 melvin-bot[bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @thesahindia (External)

melvin-bot[bot] avatar Oct 21 '22 11:10 melvin-bot[bot]

Triggered auto assignment to @srikarparsi (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

melvin-bot[bot] avatar Oct 21 '22 11:10 melvin-bot[bot]

Job posted https://www.upwork.com/jobs/~0138615bf32be2c02c

mallenexpensify avatar Oct 21 '22 23:10 mallenexpensify

Hi I saw the job on Upwork and did some digging, checked the code and everything seems to be good, unless someone pushed a staging build to the production?

Coz the environment variables are the only things that differentiates things in the process, so maybe the build pushed to the production (public) is a staging environment build?

If not, can I more context on it? I mean like access to things like developer console, from which I can find answers and most probably solution?

mayocube avatar Oct 22 '22 00:10 mayocube

did some digging, checked the code and everything seems to be good, unless someone pushed a staging build to the production?

I also feel the same. @srikarparsi, can you please look into it?

thesahindia avatar Oct 22 '22 06:10 thesahindia

I am not too sure but seems to me that it could be a caching issue. The build that gets downloaded and then that build does not match with version in package.json in the build. And then we see a badge.

I hypothesize this as I see version shown in screenshot is different from what should have been in playstore at that time ( by looking at the release in github). We had a newer build versions deployed to PROD on Oct 17 i.e 1.2.17-4 where as @mallenexpensify 's screenshot on Oct 21 says build version is 1.2.16-5. See the below.

Screen Shot 2022-10-22 at 21 46 46

I could be totally wrong but I had recently seen this code that puts tha badge. And i think because of this mismatch in versions we see that badge

Screen Shot 2022-10-22 at 21 35 21

smrutiparida avatar Oct 22 '22 16:10 smrutiparida

asked internally @thesahindia, will get back to you on if we pushed a stg build to prod

srikarparsi avatar Oct 24 '22 23:10 srikarparsi

I don't believe there was an accidental deploy, as our internal releases line up with the PlayStore builds. Instead, it seems like a regression in our versioning logic as was suggested above.

Staging version: 1.2.18 Prod version: 1.2.19-1

Julesssss avatar Oct 25 '22 07:10 Julesssss

that means there would have to be a PR recently which would break this logic, that could help narrowing this down

mountiny avatar Oct 25 '22 09:10 mountiny

Adding the deployBlockerLabel to prevent further deploys until we can confirm the cause. I know it's useless given this exists on prod, but it'll hopefully speed up the release of the eventual fix 🤞

Julesssss avatar Oct 25 '22 09:10 Julesssss

that means there would have to be a PR recently which would break this logic, that could help narrowing this down

Just want to inform that I had reported this on Aug 17, if it helps.

Puneet-here avatar Oct 25 '22 10:10 Puneet-here

This is the deploy checklist for the current prod build, which occurred on August 17th. There were two releases in quick succession on Aug 16th too (1 + 2).

Nothing stands out as a potential cause 😕

Julesssss avatar Oct 25 '22 10:10 Julesssss

Removing deploy blocker label since its not really a blocker. But keeping the hourly tag since it sounds urgent to address. I think we can also create a fire-room to discuss this.

chiragsalian avatar Oct 26 '22 19:10 chiragsalian