Add a check for percent value on upload progress to prevent app from crashing

Description

PR here, low risk: https://github.com/mattermost/mattermost-webapp/pull/2223

To discuss whether we'd take it in 5.6.1 or not. Two reports of failed file uploads

version: 5.6

Environments: unknown - Could be a specific OS/Browser issue

QA Test Steps

I don't think we will be able to verify this as we don't have replication steps

Activity

Show:
Linda Mitchell
December 26, 2018, 11:54 PM

Repeated file testing on v5.7 rctesting and Se server, including browser and desktop and RN apps. No new tests needed; marked tests done on PR. Closing.

Linda Mitchell
December 21, 2018, 8:23 AM

Will need to do the same kind of testing for v5.7.

Linda Mitchell
December 19, 2018, 9:34 PM

v5.6.1-rc3: Sudheer said none of us ever repro'ed this issue so we're unable firsthand to verify the fix, but I did a bunch of testing different file uploads, including on iOS and Android, and found no issues. Calling this good for 5.6.1.

Done
Your pinned fields
Click on the next to a field label to start pinning.

Assignee

Sudheer Timmaraju

QA Assignee

Linda Mitchell

Reporter

Jason Blais

Mattermost Team

Core Features

QA Testing Areas

Attachments