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

Stuck upload/undismissable notification #1827

Closed
roundhill opened this issue Sep 5, 2014 · 4 comments
Closed

Stuck upload/undismissable notification #1827

roundhill opened this issue Sep 5, 2014 · 4 comments
Labels
Posting/Editing [Pri] High [Type] Bug
Milestone
3.8

Comments

@roundhill
Copy link
Contributor

A user had their post upload get stuck and they couldn't remove the notification:
screenshot_2014-09-05-09-15-16
@roundhill roundhill added the bug label Sep 5, 2014
@roundhill roundhill added this to the 3.2 milestone Sep 5, 2014
@roundhill
Copy link
Contributor Author

I reproduced this by starting an upload, then switching my device from 4G/LTE to EDGE. Perhaps a timeout isn't firing?

This was referenced Sep 12, 2014
Closed
Merged
@maxme maxme reopened this Feb 20, 2015
@maxme maxme modified the milestones: 3.8, 3.2 Feb 20, 2015
@maxme
Copy link
Contributor

maxme commented Feb 20, 2015

Reopening this, 3.8 priority. Since we fixed #2282, the workaround used doesn't work (swipe the "failed" notification and try to upload again).

We must find the root issue that causes the notification and the post to get stuck in an "uploading" state.

@maxme
Copy link
Contributor

maxme commented Feb 23, 2015

  • PostUploadService must be a foreground service
  • PostUploadService.onDestroy should reset the Post status from uploading to local draft
  • use PostUploadService.onStartCommand instead PostUploadService.onStart

@maxme
Copy link
Contributor

maxme commented Feb 23, 2015

  • Notification must be created as soon as the onStartCommand is called

@maxme maxme mentioned this issue Feb 23, 2015
Merged
nbradbury added a commit that referenced this issue Feb 24, 2015
@nbradbury
Merge pull request #2365 from wordpress-mobile/issue/1827-post-stuck-…
33058b5 
…in-uploading-state

fix #1827 post stuck in uploading state
@maxme maxme closed this as completed Feb 24, 2015
nbradbury added a commit that referenced this issue Nov 25, 2015
@nbradbury
Merge pull request #2365 from wordpress-mobile/issue/1827-post-stuck-…
3f8bedb 
…in-uploading-state

fix #1827 post stuck in uploading state
oguzkocer pushed a commit that referenced this issue May 23, 2017
@nbradbury
Merge pull request #2365 from wordpress-mobile/issue/1827-post-stuck-…
682b62b 
…in-uploading-state

fix #1827 post stuck in uploading state
oguzkocer pushed a commit that referenced this issue May 23, 2017
@nbradbury
Merge pull request #2365 from wordpress-mobile/issue/1827-post-stuck-…
819ced5 
…in-uploading-state

fix #1827 post stuck in uploading state
oguzkocer pushed a commit that referenced this issue May 23, 2017
@nbradbury
Merge pull request #2365 from wordpress-mobile/issue/1827-post-stuck-…
d370d2b 
…in-uploading-state

fix #1827 post stuck in uploading state
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Posting/Editing [Pri] High [Type] Bug
Projects
None yet
Milestone
3.8
Development

No branches or pull requests
3 participants
@maxme @daniloercoli @roundhill