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

Execute actions right away instead of waiting for API call result #2016

Merged
merged 14 commits into from
Oct 3, 2024

Conversation

tevincent
Copy link
Contributor

@tevincent tevincent commented Aug 19, 2024

  • Marked as read/unread/favorite/unfavorite
    • Update locally immediately
    • Make WS call
    • If there's an error, reset to previous value
  • Move/Delete
    • Create a local variable in Realm (isMovedOutLocally) and if it is being moved, set this value to true without waiting for the API to return. When displaying the ThreadList, filter on isMovedOutLocally == false
    • If there's an error, return as before

@tevincent tevincent marked this pull request as draft August 19, 2024 15:12
@tevincent tevincent marked this pull request as ready for review August 20, 2024 06:34
Copy link

@tevincent tevincent marked this pull request as draft August 20, 2024 08:49
@KevinBoulongne KevinBoulongne self-requested a review August 20, 2024 11:10
@KevinBoulongne KevinBoulongne added the feature Add a new function or capability to the project label Aug 20, 2024
@tevincent tevincent force-pushed the instant-action branch 2 times, most recently from 23eb4ab to c090bac Compare September 27, 2024 14:11
@tevincent tevincent added the rebase Add this label to rebase the PR label Oct 1, 2024
@github-actions github-actions bot removed the rebase Add this label to rebase the PR label Oct 1, 2024
@tevincent tevincent added the rebase Add this label to rebase the PR label Oct 1, 2024
@github-actions github-actions bot removed the rebase Add this label to rebase the PR label Oct 1, 2024
@tevincent tevincent marked this pull request as ready for review October 3, 2024 08:05
@tevincent tevincent added the rebase Add this label to rebase the PR label Oct 3, 2024
@github-actions github-actions bot removed the rebase Add this label to rebase the PR label Oct 3, 2024
@tevincent tevincent added the rebase Add this label to rebase the PR label Oct 3, 2024
@github-actions github-actions bot removed the rebase Add this label to rebase the PR label Oct 3, 2024
@tevincent tevincent added the rebase Add this label to rebase the PR label Oct 3, 2024
@github-actions github-actions bot removed the rebase Add this label to rebase the PR label Oct 3, 2024
@KevinBoulongne KevinBoulongne added the rebase Add this label to rebase the PR label Oct 3, 2024
@github-actions github-actions bot removed the rebase Add this label to rebase the PR label Oct 3, 2024
Copy link

sonarqubecloud bot commented Oct 3, 2024

@tevincent tevincent merged commit c81c7cc into master Oct 3, 2024
4 checks passed
@tevincent tevincent deleted the instant-action branch October 3, 2024 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Add a new function or capability to the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants