Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[HOLD for payment 2024-03-20] HIGH: [$1000] MacOS App does not update via 'Check for Updates' when an update is available #36522

Closed
1 of 6 tasks
m-natarajan opened this issue Feb 14, 2024 · 71 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Engineering External Added to denote the issue can be worked on by a contributor Weekly KSv2

Comments

@m-natarajan
Copy link

m-natarajan commented Feb 14, 2024

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Version Number: 1.4.40-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @mallenexpensify
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1707839105781559

Action Performed:

Prereq: Must be running an out-of-date version of the MacOS app + have an update to the MacOS app available

  1. Open MacOS app
  2. Ignore any 'update available' pop up
  3. Select New Expensify > Check for Update in the menu bar
  4. You'll see a notification titled "Update Available"

Expected Result:

If an update is available, there should be an option to update the app.

Actual Result:

There isn't a way to update the app in the message. The button just says 'Sounds Good' and clicking it only closes the

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

2024-02-13_14-28-13.mp4
update.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~019cdfd41c70cb0eeb
  • Upwork Job ID: 1757798834716602368
  • Last Price Increase: 2024-03-08
Issue OwnerCurrent Issue Owner: @eVoloshchak
@m-natarajan m-natarajan added External Added to denote the issue can be worked on by a contributor Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Feb 14, 2024
Copy link

melvin-bot bot commented Feb 14, 2024

Job added to Upwork: https://www.upwork.com/jobs/~019cdfd41c70cb0eeb

@melvin-bot melvin-bot bot changed the title App not updating after clicking sounds good popup for update [$500] App not updating after clicking sounds good popup for update Feb 14, 2024
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Feb 14, 2024
Copy link

melvin-bot bot commented Feb 14, 2024

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

Copy link

melvin-bot bot commented Feb 14, 2024

Triggered auto assignment to @garrettmknight (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@garrettmknight garrettmknight added Bug Something is broken. Auto assigns a BugZero manager. and removed Bug Something is broken. Auto assigns a BugZero manager. labels Feb 14, 2024
Copy link

melvin-bot bot commented Feb 14, 2024

Triggered auto assignment to @johncschuster (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@kameshwarnayak
Copy link
Contributor

Looks like an expected behaviour. The message clearly says "The new version will be available shortly. We'll notify you when we're ready to update." It is not supposed to do anything I suppose.

@garrettmknight
Copy link
Contributor

@johncschuster just reassigning in case this needs help while I'm OOO this week - will be back 2/21 to pick it back up

@mallenexpensify
Copy link
Contributor

mallenexpensify commented Feb 16, 2024

@johncschuster and @eVoloshchak , are you able to reproduce this on Desktop? It's still happening for me, trying to suss out how common the bug is (and what the specific work-around is to get the app to update).

Also checking in the Slack thread to see if others are experiencing and repro steps if so.

@dangrous
Copy link
Contributor

I think this is happening to me on Desktop staging - v 1.4.40-0

@melvin-bot melvin-bot bot added the Overdue label Feb 19, 2024
@eVoloshchak
Copy link
Contributor

I'm able to reproduce this on Desktop
I agree with @kameshwarnayak's comment, the modal itself should just close and do nothing

The message clearly says "The new version will be available shortly. We'll notify you when we're ready to update." It is not supposed to do anything I suppose.
(what is the purpose of this modal?)

However, I think the real problem is the app not updating. My Desktop app is stuck on 1.4.8-1, pressing 'Update New Expensify' does nothing

@melvin-bot melvin-bot bot removed the Overdue label Feb 20, 2024
Copy link

melvin-bot bot commented Feb 20, 2024

@garrettmknight, @johncschuster, @eVoloshchak Eep! 4 days overdue now. Issues have feelings too...

@dangrous
Copy link
Contributor

yeah thanks for the clarification @eVoloshchak - I think usually there is some delay between clicking that and the update itself, so the update is the issue not the button.

I am struggling to figure out a way to fix stuck versions, though. Because even if we fix this, how do we update the versions that won't update in order for them to update?

@mallenexpensify
Copy link
Contributor

The message clearly says "The new version will be available shortly. We'll notify you when we're ready to update." It is not supposed to do anything I suppose.
(what is the purpose of this modal?)

@kameshwarnayak , @eVoloshchak , For the past couple years, the flow was a version of

  • Click Check for Updates
  • Click button to Update
  • Pop surfaces asking if you'd like to update
  • App closes, updates, reopens.

It seemed to make more sense that way than the way it is now where the version can get stuck and not updated for a while. Albeit... this is only for Desktop and I think most users are on web, so it's likely not the highest priority.

Copy link

melvin-bot bot commented Feb 21, 2024

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

@garrettmknight
Copy link
Contributor

Back and picking this one up - @mallenexpensify I think this is might be working as expected. When an update is available, the menu item reads something differently than 'Check for updates'. I think it's 'Update New Expensify' or something like that. When you click it, it updates.

To make sure I'm repro'ing correctly, what's the initial scenario we need to test?

  • Must have updated NewDot Desktop version available
  • Must have ignored the initial pop-up? Or is this before the pop-up happens?

@garrettmknight garrettmknight added the Needs Reproduction Reproducible steps needed label Feb 21, 2024
@garrettmknight
Copy link
Contributor

Setting to Needs reproduction until I can figure out how to test.

@puneetlath
Copy link
Contributor

I'm experiencing this problem too.

image

You can click "check for updates" and then "sounds good" as much as I want. The ability to actually update the app never arrives, even though I am behind.

@garrettmknight garrettmknight changed the title HIGH: [$1000] MacOS App does not update via 'Check for Updates' when an update is available [HOLD for #37615] HIGH: [$1000] MacOS App does not update via 'Check for Updates' when an update is available Mar 18, 2024
@garrettmknight garrettmknight added Daily KSv2 and removed Weekly KSv2 labels Mar 18, 2024
@garrettmknight
Copy link
Contributor

I just tested and 'Check for updates' didn't notify me when the download was complete. I waited for 30 mins before closing/reopening the app to confirm that the udpate completed. @s77rt should that change have gone into effect immediately or do I need to wait for another release to test again?

@s77rt
Copy link
Contributor

s77rt commented Mar 19, 2024

@garrettmknight On which version you tested? You need at least version 1.4.54-0 to test on

@garrettmknight
Copy link
Contributor

I'm on 1.4.54-1 now - will wait for the next update to test in that case.

@lakchote
Copy link
Contributor

lakchote commented Mar 20, 2024

@garrettmknight On which version you tested? You need at least version 1.4.54-0 to test on

@s77rt It should have worked for Garrett, no? Since he was on version 1.4.54-1.

It has now been deployed to production on 1.4.54-4 @garrettmknight, could you test again please?

@garrettmknight
Copy link
Contributor

I wasn't on 1.4.54-1 when I tested, that's what it updated to from the older version. Annnnnddd my computer restarted so the new version installed automatically when I reopened. WIll hold to test on the next release - unless you have an earlier version and can test now @lakchote ?

@puneetlath
Copy link
Contributor

I was just able to update my Desktop app from 1.4.54-1 to 1.4.55-0.

@lakchote
Copy link
Contributor

I wasn't on 1.4.54-1 when I tested, that's what it updated to from the older version. Annnnnddd my computer restarted so the new version installed automatically when I reopened. WIll hold to test on the next release - unless you have an earlier version and can test now @lakchote ?

Understood, thanks for the precision Garrett. Unfortunately, I already have the latest version.

But it looks like it works since @puneetlath just test it successfully!

@garrettmknight
Copy link
Contributor

Nice, thanks @puneetlath. We can finish this one up then. Since this got fixed without the work here, but we had a pretty active PR going I'm going to partially pay out at 50%.

@garrettmknight garrettmknight changed the title [HOLD for #37615] HIGH: [$1000] MacOS App does not update via 'Check for Updates' when an update is available [HOLD for payment 2024-03-20] HIGH: [$1000] MacOS App does not update via 'Check for Updates' when an update is available Mar 20, 2024
@garrettmknight garrettmknight added Awaiting Payment Auto-added when associated PR is deployed to production and removed Reviewing Has a PR in review labels Mar 20, 2024
@garrettmknight
Copy link
Contributor

@s77rt offer out to you.

@garrettmknight
Copy link
Contributor

garrettmknight commented Mar 20, 2024

Summary of payment:

Upwork link

@s77rt
Copy link
Contributor

s77rt commented Mar 20, 2024

@puneetlath Just to confirm, you saw a popup or a notification informing you that the update is downloaded right? Because the update was never broken, it was just silent

@s77rt
Copy link
Contributor

s77rt commented Mar 20, 2024

@garrettmknight Accepted

@puneetlath
Copy link
Contributor

Hmm, I don't remember if I saw the popup 😅. I can check again next time. When I do "check for updates" now I get this, which I wasn't getting before, so that seems good.

Screenshot 2024-03-20 at 10 03 05 AM

Copy link

melvin-bot bot commented Mar 20, 2024

Payment Summary

Upwork Job

BugZero Checklist (@garrettmknight)

  • I have verified the correct assignees and roles are listed above and updated the neccesary manual offers
  • I have verified that there are no duplicate or incorrect contracts on Upwork for this job (https://www.upwork.com/ab/applicants/1757798834716602368/hired)
  • I have paid out the Upwork contracts or cancelled the ones that are incorrect
  • I have verified the payment summary above is correct

@garrettmknight
Copy link
Contributor

Will complete the payment summary tomorrow when I test to double-confirm things are working.

@garrettmknight
Copy link
Contributor

garrettmknight commented Mar 21, 2024

Noting that I just clicked check for updates and got the update available message + now I can't click check for updates:

Screenshot 2024-03-21 at 3 15 46 PM

@garrettmknight
Copy link
Contributor

And just got the pop-up:
Screenshot 2024-03-21 at 3 16 02 PM

@garrettmknight garrettmknight added Weekly KSv2 and removed Daily KSv2 labels Mar 21, 2024
@garrettmknight
Copy link
Contributor

@eVoloshchak please request payment when you're ready.

@JmillsExpensify
Copy link

$500 approved for @eVoloshchak based on summary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Engineering External Added to denote the issue can be worked on by a contributor Weekly KSv2
Projects
No open projects
Status: CRITICAL
Development

No branches or pull requests