[PR #315] [MERGED] Add information for uploading asset and error indication with error message for each failed upload. #8556

Closed
opened 2026-02-05 13:49:18 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/immich-app/immich/pull/315
Author: @alextran1502
Created: 7/6/2022
Status: Merged
Merged: 7/7/2022
Merged by: @alextran1502

Base: mainHead: dev/add-info-box-for-upload-asset


📝 Commits (10+)

  • d83496f Added info box
  • fef6e1d Fixed upload endpoin doesn't report error status code
  • 3826440 Added chip to show update error
  • a396359 Added chip to show failed upload
  • 86fe3be Add duplication check for upload
  • 69f1139 Better duplication checking placement
  • 4dd89d8 Up version
  • 481d494 Remove check for duplicated asset
  • d31d149 Added failed backup status route
  • acf7601 added page

📊 Changes

19 files changed (+706 additions, -87 deletions)

View changed files

📝 mobile/ios/fastlane/Fastfile (+1 -1)
📝 mobile/lib/modules/backup/models/backup_state.model.dart (+42 -16)
mobile/lib/modules/backup/models/check_duplicate_asset_response.model.dart (+48 -0)
mobile/lib/modules/backup/models/current_upload_asset.model.dart (+78 -0)
mobile/lib/modules/backup/models/error_upload_asset.model.dart (+53 -0)
📝 mobile/lib/modules/backup/providers/backup.provider.dart (+38 -6)
mobile/lib/modules/backup/providers/error_backup_list.provider.dart (+23 -0)
📝 mobile/lib/modules/backup/services/backup.service.dart (+63 -9)
📝 mobile/lib/modules/backup/views/backup_controller_page.dart (+161 -28)
mobile/lib/modules/backup/views/failed_backup_status_page.dart (+139 -0)
📝 mobile/lib/routing/router.dart (+6 -0)
📝 mobile/lib/routing/router.gr.dart (+20 -1)
📝 mobile/lib/shared/services/network.service.dart (+1 -1)
📝 mobile/pubspec.yaml (+1 -1)
📝 server/apps/immich/src/api-v1/asset/asset.controller.ts (+12 -11)
📝 server/apps/immich/src/api-v1/asset/asset.service.ts (+8 -10)
server/apps/immich/src/api-v1/asset/dto/check-duplicate-asset.dto.ts (+9 -0)
📝 server/apps/immich/src/constants/server_version.constant.ts (+2 -2)
📝 web/src/lib/utils/file-uploader.ts (+1 -1)

📄 Description

No description provided


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/immich-app/immich/pull/315 **Author:** [@alextran1502](https://github.com/alextran1502) **Created:** 7/6/2022 **Status:** ✅ Merged **Merged:** 7/7/2022 **Merged by:** [@alextran1502](https://github.com/alextran1502) **Base:** `main` ← **Head:** `dev/add-info-box-for-upload-asset` --- ### 📝 Commits (10+) - [`d83496f`](https://github.com/immich-app/immich/commit/d83496fa519a04d7e9e3d1fc9c5b02656f5ce419) Added info box - [`fef6e1d`](https://github.com/immich-app/immich/commit/fef6e1dd786ccfb5f3e9a9366349614c0d9b1164) Fixed upload endpoin doesn't report error status code - [`3826440`](https://github.com/immich-app/immich/commit/38264400f325e2acf219bc9807b460915181bdb7) Added chip to show update error - [`a396359`](https://github.com/immich-app/immich/commit/a3963593e367ffd737ed15f23c4999c86eea68c9) Added chip to show failed upload - [`86fe3be`](https://github.com/immich-app/immich/commit/86fe3be0e1bbf0a233f957d20e59ab063702d610) Add duplication check for upload - [`69f1139`](https://github.com/immich-app/immich/commit/69f11391676fa06d573e2a9ca21f7c0b6070fed5) Better duplication checking placement - [`4dd89d8`](https://github.com/immich-app/immich/commit/4dd89d873bffcd0498ab7337c4d7665785405b8d) Up version - [`481d494`](https://github.com/immich-app/immich/commit/481d49441d74f841b854abd2fddc57cad57be147) Remove check for duplicated asset - [`d31d149`](https://github.com/immich-app/immich/commit/d31d149f12fe1ca0623b189642e57def05524043) Added failed backup status route - [`acf7601`](https://github.com/immich-app/immich/commit/acf7601a36088f8a85fe299072fa52ac5786b86a) added page ### 📊 Changes **19 files changed** (+706 additions, -87 deletions) <details> <summary>View changed files</summary> 📝 `mobile/ios/fastlane/Fastfile` (+1 -1) 📝 `mobile/lib/modules/backup/models/backup_state.model.dart` (+42 -16) ➕ `mobile/lib/modules/backup/models/check_duplicate_asset_response.model.dart` (+48 -0) ➕ `mobile/lib/modules/backup/models/current_upload_asset.model.dart` (+78 -0) ➕ `mobile/lib/modules/backup/models/error_upload_asset.model.dart` (+53 -0) 📝 `mobile/lib/modules/backup/providers/backup.provider.dart` (+38 -6) ➕ `mobile/lib/modules/backup/providers/error_backup_list.provider.dart` (+23 -0) 📝 `mobile/lib/modules/backup/services/backup.service.dart` (+63 -9) 📝 `mobile/lib/modules/backup/views/backup_controller_page.dart` (+161 -28) ➕ `mobile/lib/modules/backup/views/failed_backup_status_page.dart` (+139 -0) 📝 `mobile/lib/routing/router.dart` (+6 -0) 📝 `mobile/lib/routing/router.gr.dart` (+20 -1) 📝 `mobile/lib/shared/services/network.service.dart` (+1 -1) 📝 `mobile/pubspec.yaml` (+1 -1) 📝 `server/apps/immich/src/api-v1/asset/asset.controller.ts` (+12 -11) 📝 `server/apps/immich/src/api-v1/asset/asset.service.ts` (+8 -10) ➕ `server/apps/immich/src/api-v1/asset/dto/check-duplicate-asset.dto.ts` (+9 -0) 📝 `server/apps/immich/src/constants/server_version.constant.ts` (+2 -2) 📝 `web/src/lib/utils/file-uploader.ts` (+1 -1) </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 13:49:18 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: immich-app/immich#8556