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.

Sign up for GitHub

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

Jump to bottom

App syncing in background #4167

Closed
ouchadam opened this issue Oct 5, 2021 · 2 comments · Fixed by #4225
Closed

App syncing in background #4167

ouchadam opened this issue Oct 5, 2021 · 2 comments · Fixed by #4225
Labels
A-Sync T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Milestone
Sprint - Element ...

Comments

@ouchadam
Copy link
Contributor

ouchadam commented Oct 5, 2021

Steps to reproduce

  1. Force stop the app / swipe from the recents menu
  2. Receive a push notification
  3. Notice the incremental sync is now acting as if the app was in the foreground

What happened?

What did you expect?

For the app to not run in the background

What happened?

After receiving a push notification whilst the application has been killed / force stopped the incremental sync starts polling

Your phone model

Pocofone f1

Operating system version

android 10 (MIUI 12.0.3.0)

Application version and app store

Latest from develop a0d5618

Homeserver

matrix.org

Have you submitted a rageshake?

No
@ouchadam ouchadam added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label Oct 5, 2021
@ouchadam ouchadam changed the title App syncing in background after receiving a push after the application has been killed App syncing in background Oct 5, 2021
@bmarty bmarty added this to the Sprint - Element 1.3.3 milestone Oct 5, 2021
@ouchadam
Copy link
Contributor Author

ouchadam commented Oct 6, 2021

I suspect the issue is tied to https://github.com/vector-im/element-android/blob/develop/vector/src/main/java/im/vector/app/VectorApplication.kt#L181

when starting the application from a push, we never enter the foreground or background as no activities are launched

@ouchadam ouchadam mentioned this issue Oct 12, 2021
Merged
@bmarty
Copy link
Member

bmarty commented Oct 12, 2021

Fixed in 1.3.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Sync T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Projects
None yet
Milestone
Sprint - Element 1.3.4
Development

Successfully merging a pull request may close this issue.

Fixing unexpected sync occurring in background element-hq/element-android
2 participants
@ouchadam @bmarty