Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TW-1760: Create UploadManager to manage upload file #1896

Merged
merged 20 commits into from
Jul 9, 2024

Conversation

nqhhdev
Copy link
Member

@nqhhdev nqhhdev commented Jun 25, 2024

Ticket

Need merge:

DoD:

UploadManager

  • UploadManager to manage upload file
  • Send fake event first
  • Add task to queue
  • Handle upload file success
  • Error handling
  • Write unit test

Improve UI/UX upload load files (Mobile/web)

  • Display cancel token and progress upload
  • Update correctly uploading file
  • Cancel uploading file => remove fake event
  • Error handling upload fail

Improve UI/UX upload media (Mobile/web)

  • Gen and upload thumbnail
  • Update correctly uploading video/image
  • Display cancel token and progress upload
  • Cancel uploading file => remove fake event
  • Error handling upload fail

Resolved

Mobile

  • Upload file
Simulator.Screen.Recording.-.iPhone.15.-.2024-06-27.at.12.09.44.mp4
Simulator.Screen.Recording.-.iPhone.15.-.2024-06-27.at.12.04.24.mp4
  • Upload media
mobile_media_1.mp4
mobile_media_2.mp4

WEB

  • Upload file
Screen.Recording.2024-06-28.at.16.25.31.mov
Screen.Recording.2024-06-28.at.16.27.05.mov
Screen.Recording.2024-06-28.at.16.29.47.mov

@nqhhdev nqhhdev changed the base branch from main to feature/upload_file June 25, 2024 10:31
Copy link

This PR has been deployed to https://linagora.github.io/twake-on-matrix/1896

@nqhhdev nqhhdev force-pushed the TW-1760-create-upload-manager branch from 3940746 to 0f94c1f Compare June 26, 2024 16:41
@nqhhdev nqhhdev force-pushed the TW-1760-create-upload-manager branch from f6a3ba7 to 61ab3a4 Compare June 27, 2024 04:31
@nqhhdev nqhhdev changed the title [WIP] TW-1760: Create UploadManager to manage upload file TW-1760: Create UploadManager to manage upload file Jul 2, 2024
@Te-Z Te-Z merged commit 50d29ef into feature/upload_file Jul 9, 2024
3 checks passed
@Te-Z Te-Z deleted the TW-1760-create-upload-manager branch July 9, 2024 14:39
nqhhdev added a commit that referenced this pull request Jul 11, 2024
* TW-1758: Create method upload file use `dio`

* TW-1758: Refactor and add cancel token and progress upload on mobile

* TW-1760: Create `UploadWorkerQueue`

* TW-1760: Create models for upload file info

* TW-1760: Define state for upload manager

* TW-1760: Support clear task in queue

* TW-1760: Update send progress for uploda in `media_api`

* TW-1760: Update state for upload

* TW-1760: Create `UploadManager` to handle upload file and media on mobile

* TW-1760: Handle UI upload file and media on mobile

* TW-1760: Upgrade matrix sdk

* TW-1760: Improve UI for upload media on mobile

* TW-1760: Update upload api on web

* TW-1760: Remove upload interactor on web not use

* TW-1760: Improve upload on web

* TW-1760: Integrate `UploadManager` for UI on web

* TW-1760: Write test case for `clearTaskInQueue` func in WorkerQueue

* TW-1760: Write unit test for uploap file on web

* TW-1760: Remove `interactor` not use

* TW-1760: Handle error from send file dialog
Te-Z pushed a commit that referenced this pull request Jul 12, 2024
* TW-1758: Create upload file on web use dio (#1765)

* TW-1758: Create method upload file use `dio`

* TW-1758: Refactor and add cancel token and progress upload on mobile

* TW-1759: Improve func convert stream to bytes (#1893)

* TW-1759: Improve func toUint8List convert stream to bytes

* TW-1759: Write unit test for func

* TW-1760: Create `UploadManager` to manage upload file (#1896)

* TW-1758: Create method upload file use `dio`

* TW-1758: Refactor and add cancel token and progress upload on mobile

* TW-1760: Create `UploadWorkerQueue`

* TW-1760: Create models for upload file info

* TW-1760: Define state for upload manager

* TW-1760: Support clear task in queue

* TW-1760: Update send progress for uploda in `media_api`

* TW-1760: Update state for upload

* TW-1760: Create `UploadManager` to handle upload file and media on mobile

* TW-1760: Handle UI upload file and media on mobile

* TW-1760: Upgrade matrix sdk

* TW-1760: Improve UI for upload media on mobile

* TW-1760: Update upload api on web

* TW-1760: Remove upload interactor on web not use

* TW-1760: Improve upload on web

* TW-1760: Integrate `UploadManager` for UI on web

* TW-1760: Write test case for `clearTaskInQueue` func in WorkerQueue

* TW-1760: Write unit test for uploap file on web

* TW-1760: Remove `interactor` not use

* TW-1760: Handle error from send file dialog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants