[BR-2136]: surface failed uploads in the retry modal and task logger - #2076
[BR-2136]: surface failed uploads in the retry modal and task logger#2076terrerox wants to merge 1 commit into
Conversation
Retry entries are grouped under their parent folder task via relatedTaskId, and non-retryable files show a "Not allowed" label (added to all locales) instead of a retry button. Also normalizes the two RetryableTask param shapes in TaskToRetryItem behind a getDisplayData helper (file names now display with their extension), tightens the retry modal row padding, and adds sizeClassName/iconSize props to TaskLoggerButton.
Deploying drive-web with
|
| Latest commit: |
af24b14
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://f7a215bf.drive-web.pages.dev |
| Branch Preview URL: | https://feat-failed-uploads-retry-ui.drive-web.pages.dev |
|
| return haveWarnings ? ( | ||
| <TaskLoggerButton onClick={infoAction} Icon={InfoIcon} iconSize={20} /> | ||
| ) : ( | ||
| <TaskLoggerButton onClick={magnifyingAction} Icon={MagnifyingGlass} /> |
There was a problem hiding this comment.
If there’s any file with error, do we no longer display the magnifying glass icon to navigate to the folder?
| const getDisplayData = (params: RetryableTask['params']): DisplayData => ({ | ||
| name: params?.filecontent?.name ?? params.plainName ?? params.name, | ||
| type: params?.filecontent?.type ?? params.type, | ||
| size: params?.filecontent?.size ?? params.size, | ||
| modifiedAt: params?.filecontent?.content.lastModified ?? params.updatedAt, | ||
| isFolder: Boolean(params?.isFolder), | ||
| }); |
There was a problem hiding this comment.
I understand that this depends on whether it’s an upload or a download, as the data arrives with the fields structured differently, but this needs to be reorganised more effectively, as looking at it like this, we can’t tell why there are so many fallback options for the same field.
Looking at this code alone, it’s impossible to tell why, for ‘name’, it checks ‘filecontent.name’, then uses ‘params.plainname’ as a fallback, and then ‘params.name’.
2 participants
Description
Retry entries are grouped under their parent folder task via relatedTaskId, and non-retryable files show a "Not allowed" label (added to all locales) instead of a retry button.
Also normalizes the two RetryableTask param shapes in TaskToRetryItem behind a getDisplayData helper (file names now display with their extension), tightens the retry modal row padding, and adds sizeClassName/iconSize props to TaskLoggerButton.
Related Issues
Related Pull Requests
Checklist
Testing Process
Additional Notes