Derek Blank

Results 22 comments of Derek Blank

> The only problematic part is the following, which I'm not sure what it does or why: > > ```swift > case .thumbnailReady(let url) where ReachabilityUtils.isInternetReachable() && media.remoteStatus == .failed:...

> The part about the editor events is what I do not completely understand In terms of what already exists and what needs to change, I think we all understand...

@crazytonyli This issue appears to be escalating in [Sentry reports](https://a8c.sentry.io/issues/4616972167/events/5ceaa34ecd704301bff28738be4539e3/?project=1438083) since the release of 23.6.0.1: ``` Error Domain=PostEditor.OverwritePost Code=1 "(null)" ``` The stack trace points to [PostService.checkLatestRevision](https://github.com/wordpress-mobile/WordPress-iOS/blob/77d72c0d1210283e301822974e4652a80561dcfd/WordPress/Classes/Services/PostService%2BRevisions.swift#L34). Would you be...

> The new "PostEditor.OverwritePost" error events are merely a way to detect and surface the underlying issue where app's post changes override desktop changes. Gotcha, thank you clarifying. During this...

@dcalhoun Would a corresponding Gutenberg Mobile PR be expected to pair with this PR? I see that the Automation label was not included. Just curious if that was intentional based...

Deferring to 24.2 milestone to evaluate this event further.

I collected all the reports of [JniAndroid$UncaughtExceptionException](https://a8c.sentry.io/issues/?project=5716771&project=1438083&project=5731682&project=1438088&query=JniAndroid%24UncaughtExceptionException&referrer=issue-list&statsPeriod=90d) from Sentry and linked all of the ones I could find that included `Resources$NotFoundException` that seemed related: | Issue | Events | Users...

@fluiddot / @antonis We've noted that the horizontal media progress indicators within the Editor no longer increment in WP/JPAndroid 24.4-rc-1. This appears to affect Android only, and has been partially...

@antonis Thanks for the extra info and context -- it helps to rule things out. 🙇 Especially, noting that https://github.com/wordpress-mobile/gutenberg-mobile/issues/6667 preceded the stories removal PRs, which I hadn't yet noted....

> I had a 2nd look on how the video upload progress is communicated and noticed that the [progress is passing to the gluecode](https://github.com/WordPress/gutenberg/blob/trunk/packages/react-native-bridge/android/react-native-bridge/src/main/java/org/wordpress/mobile/WPAndroidGlue/WPAndroidGlueCode.java#L1089). Thanks for taking a second look,...