[Mobile]Fix image upload progress not being displayed on the second time #14799
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
To fix: wordpress-mobile/gutenberg-mobile#809
I had updated the upload listener to get started on componentDidMount and get removed at componentDidMount. Comparing to previous, new version will sometimes listen to upload events unnecessarily, but if I am not missing sth that's also the case when we have multiple image blocks uploading in parallel, we'll have some unrelated mediaUpload events coming in that case too. We trust the below check to ignore complications, it looked to me as we can lean on this and use a simpler logic for adding&removing listener for media upload events. Please correct me if I am missing sth because I might be :)
How has this been tested?
Tested with Parent PR: wordpress-mobile/gutenberg-mobile#811
Screenshots
Types of changes
Checklist: