Skip to content

Commit

Permalink
Update OfflineUX.md
Browse files Browse the repository at this point in the history
Co-authored-by: Yuwen Memon <yuwen@expensify.com>
  • Loading branch information
Luke9389 and yuwenmemon committed Jun 3, 2022
1 parent 3c7c639 commit e2e9fb7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion OfflineUX.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ When a user clicks the pin button <img style="height: 10px; width: 10px;" src=".

2. **UI Changes:** The chat should go to the top of the list with the other pinned chats, and the pin button should look darker than it did before. This is visual feedback that clicking the pin button worked.

If the user is offline, we don't need to wait for the API request to finish before doing all that visual stuff because this particular API request has almost no way of failing, and we know what the server will return in advance. That means we can safely assume that when we send the command, it will succeed and that's why we let the user continue using the app as if the action succeeded.
If the user is offline, we don't need to wait for the API request to finish before doing all that visual stuff because this particular API request has almost no way of failing, and we know what the server will return in advance. That means we can safely assume that when we retry the command, it will succeed and that's why we let the user continue using the app as if the action succeeded.

<hr />

Expand Down

0 comments on commit e2e9fb7

Please sign in to comment.