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] [$250] IOS - Settings - Changes displayed after closing and reopen app #12355

Closed
kbecciv opened this issue Nov 1, 2022 · 16 comments
Closed
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 External Added to denote the issue can be worked on by a contributor

Comments

@kbecciv
Copy link

kbecciv commented Nov 1, 2022

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


Issue found when executing PR #12125

Action Performed:

  1. Launch the app
  2. Log in any account
  3. Navigate to Settings > Profile
  4. Disable the internet connection in the device
  5. Modify first name, last name, and your timezone
  6. Save and verify everything looks like it updated correctly
  7. Click back button to go back to settings page
  8. Click profile and verify changes are preserved
  9. Click back button to go back to settings page again
  10. Go back online
  11. Click on the Profile and check

Expected Result:

Changes should be displayed after go back online.

Actual Result:

Changes displayed after closing and reopen app

Workaround:

Unknown

Platform:

Where is this issue occurring?

  • iOS

Version Number: 1.2.22.1

Reproducible in staging?: Yes

Reproducible in production?: Yes

Email or phone of affected tester (no customers):

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

RPReplay_Final1667327839.MP4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

@kbecciv kbecciv added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Nov 1, 2022
@melvin-bot
Copy link

melvin-bot bot commented Nov 1, 2022

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

@melvin-bot
Copy link

melvin-bot bot commented Nov 7, 2022

@flaviadefaria Eep! 4 days overdue now. Issues have feelings too...

@flaviadefaria
Copy link
Contributor

Was able to recreate the bug so will create UW job posts.

@melvin-bot melvin-bot bot removed the Overdue label Nov 8, 2022
@flaviadefaria flaviadefaria added the External Added to denote the issue can be worked on by a contributor label Nov 8, 2022
@melvin-bot
Copy link

melvin-bot bot commented Nov 8, 2022

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

@melvin-bot
Copy link

melvin-bot bot commented Nov 8, 2022

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

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Nov 8, 2022
@melvin-bot
Copy link

melvin-bot bot commented Nov 8, 2022

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

@melvin-bot melvin-bot bot changed the title IOS - Settings - Changes displayed after closing and reopen app [$250] IOS - Settings - Changes displayed after closing and reopen app Nov 8, 2022
@flaviadefaria
Copy link
Contributor

@mountiny mountiny changed the title [$250] IOS - Settings - Changes displayed after closing and reopen app [HOLD] [$250] IOS - Settings - Changes displayed after closing and reopen app Nov 8, 2022
@mountiny
Copy link
Contributor

mountiny commented Nov 8, 2022

Putting this on Hold as I tihnk there is already issue for the settings not being saved correctly

@mountiny mountiny removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Nov 8, 2022
@mountiny
Copy link
Contributor

mountiny commented Nov 8, 2022

This is indeed just a regression from #11632

@mountiny
Copy link
Contributor

mountiny commented Nov 8, 2022

I will close this one in favour of the original #11632!

@jasperhuangg
Copy link
Contributor

Tried the reproduction steps on dev and it appears to be working, so this might be fixed with the next App deploy, stay tuned..

Screen.Recording.2022-11-08.at.8.35.21.AM.mov

@flaviadefaria
Copy link
Contributor

flaviadefaria commented Nov 8, 2022

Cool let's wait for the next deploy before I delete the UW posts.

@mvtglobally
Copy link

Issue not reproducible during KI retests. (First week)

@mountiny
Copy link
Contributor

mountiny commented Nov 9, 2022

Also wasnt able to repro on the latest Staging iOS. @jasperhuangg seems like caught it right.

@mountiny
Copy link
Contributor

We have not been able to reproduce this issue in staging and production anymore. The values save correctly, although in some testing tries the old values flickers/appears temporarily in the product before the correct saved values settle.

This is most likely related to order of the read/write requests which go to the server once we get online #12219

@flaviadefaria
Copy link
Contributor

Closed the UW post!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 External Added to denote the issue can be worked on by a contributor
Projects
None yet
Development

No branches or pull requests

6 participants