[$500] `Switch to focus mode` modal appears every time sign out and sign in for same account

[m-natarajan]

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.46-0
Reproducible in staging?: y
Reproducible in production?: y
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: @iwiznia
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1709229575556299

Action Performed:

1. Have an account with many chats
2. Have your message Priority mode setting set tomost recent mode
3. Sign out and sign back in
4. See the "switch to focus mode" modal is shown and you are switched to focus mode
5. Switch your message Priority mode setting tomost recent mode
6. Sign out and sign back in

Expected Result:

The account's message Priority mode should not be switched to #focus mode, nor should the #focus mode modal appear since the account should not be in #focus mode.

Actual Result:

The switch to focus mode modal is shown and you are switched to focus mode

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

Recording.2808.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01f91501e19c2b41b5
• Upwork Job ID: 1765136631800881152
• Last Price Increase: 2024-03-05
• Automatic offers:
• ishpaul777 | Reviewer | 0
• bernhardoj | Contributor | 0

Issue Owner

Current Issue Owner: @johncschuster

[melvin-bot]

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

[Krishna2323]

Proposal

Please re-state the problem that we are trying to solve in this issue.

Switch to focus mode modal appears every time sign out and sign in for same account

What is the root cause of that problem?

We don't update NVP_TRY_FOCUS_MODE Onyx value, instead we just assign it to a varibale.

App/src/libs/actions/PriorityMode.ts

Lines 146 to 147 in 48657b4

	// Record that we automatically switched them so we don't ask again.
	hasTriedFocusMode = true;

What changes do you think we should make in order to solve the problem?

We need to update it to Onyx.merge(ONYXKEYS.NVP_TRY_FOCUS_MODE, true);.

Result

[dukenv0307]

Proposal

Please re-state the problem that we are trying to solve in this issue.

• Switch to focus mode modal appears every time sign out and sign in for same account

What is the root cause of that problem?

• Calling API UpdateChatPriorityMode to set the focus mode will also set tryFocusMode: true (by BE `s response). But when singing out, we clear tryFocusMode key, that leads to the condition here return false, so the switch to focus mode modal appears.

What changes do you think we should make in order to solve the problem?

• We can keep the tryFocusMode when signing out. For example, add tryFocusMode key to this

What alternative solutions did you explore? (Optional)

• When user signs in, BE can return tryFocusMode

Update:

Currently, BE return nvp_tryFocusMode after sigining in. So we can use this key rather than tryFocusMode

Improvement:

• We are calling the tryFocusModeUpdate with 300ms delay:
  

  App/src/libs/actions/PriorityMode.ts

  Line 37 in 0c09803

  const autoSwitchToFocusMode = debounce(tryFocusModeUpdate, 300, {leading: true});

• This can lead to the case that focus mode is not set if the reports data is not fetched successfully when tryFocusModeUpdate is called. We can improve it as we did with onboard modal here. Below is the detail:

1. In here, create a useEffect that its dependences are whatever the tryFocusModeUpdate depends on. It will be something as:

    useEffect(() => {
        tryFocusModeUpdate(reports, hasTriedFocusMode, ...);
    }, [isLoadingApp]);

2. Then update the withOnyx in here to subscribe to the above data

3. Update the tryFocusModeUpdate to receive the reports, hasTriedFocusMode, ... as params

[melvin-bot]

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

[CortneyOfstad]

Hi @johncschuster! This issue was assigned to me after I went OoO last night, so reassigning. Thanks!

[bernhardoj]

Proposal

Please re-state the problem that we are trying to solve in this issue.

Switch to focus mode modal appears and the priority mode is switched to focus every time a user signs out and signs in back.

What is the root cause of that problem?

In tryFocusModeUpdate, we check for hasTriedFocusMode value. If it's true, we won't switch it to focus mode. However, after sign-in, hasTriedFocusMode is null. We already guard the function to wait until all required data is available,

App/src/libs/actions/PriorityMode.ts

Lines 111 to 112 in af68510

	function tryFocusModeUpdate() {
	isReadyPromise.then(() => {

but we only check whether hasTriedFocusMode is undefined or not.

App/src/libs/actions/PriorityMode.ts

Lines 103 to 109 in af68510

	function checkRequiredData() {
	if (allReports === undefined || hasTriedFocusMode === undefined || isInFocusMode === undefined || isLoadingReportData) {
	return;
	}
	resolveIsReadyPromise();
	}

The hasTriedFocusMode value could be null because it's cleared when we sign out. That's why the type is either boolean, undefined, or null.

What changes do you think we should make in order to solve the problem?

In checkRequiredData, return early if hasTriedFocusMode is undefined or null.

Or even better, we can just check if hasTriedFocusMode is not a boolean.

[melvin-bot]

@johncschuster Whoops! This issue is 2 days overdue. Let's get this updated quick!