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.

Sign up for GitHub

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

Jump to bottom

PE-4960: Release ArDrive App v2.24.1 #1469

Merged
merged 15 commits into from
Nov 7, 2023
Merged

PE-4960: Release ArDrive App v2.24.1 #1469

merged 15 commits into from
Nov 7, 2023

Conversation

thiagocarvalhodev
Copy link
Collaborator

Release Notes

  • Fixes the issue where the upload progress remained at 100% during uploads for some users.

thiagocarvalhodev and others added 15 commits October 25, 2023 16:27
@thiagocarvalhodev
refactor
6b263bd 
- refactors how we update the progress of the upload
@thiagocarvalhodev
chore: remove noisy logs
7625ec9
@thiagocarvalhodev
refactor(upload controller)
cc0e4d8 
- refactors the class moving the most important class at the top of the file
- refactors the method to update the upload progress
- removes a unused class
@thiagocarvalhodev
refactor(upload controller)
c23d188 
- Save the file on database when its upload finishes.
- Introduces the `onCompleteTask` API on `UploadController`
@thiagocarvalhodev
Merge branch 'dev' into performance-improvements-on-updating-upload-p…
fd2ade0 
…rogress
@thiagocarvalhodev
Update data_bundler.dart
2d3d148
@thiagocarvalhodev
Update data_bundler.dart
2b860fd 
remove noisy prints
@thiagocarvalhodev
Update upload_controller.dart
e313ee6 
refactor the total size calculation
@thiagocarvalhodev
fix(uploader)
ea7f8e7 
- fixes a memory leak on web uploads using Turbo
- blocks the UI from update while uploading files, and updates in the end of the upload
@thiagocarvalhodev
fix(lint and test): add the mock for activitiy tracker
b92a342
@thiagocarvalhodev
chore: remove print statements and polyfill
93023f1
@thiagocarvalhodev
Update pubspec.yaml
eaa5468 
bump pubspec version
@thiagocarvalhodev
Merge pull request #1460 from ardriveapp/PE-4892-refactor-the-progres…
9fcbc1b 
…s-update-uploading-files

PE-4892: refactor the progress update uploading files
@thiagocarvalhodev
add release notes
39d132f
@thiagocarvalhodev
Merge pull request #1468 from ardriveapp/PE-4960-release-ar-drive-app…
40c9c02 
…-v-2-24-1

PE-4960: pre-release tasks
@thiagocarvalhodev thiagocarvalhodev self-assigned this Nov 7, 2023
matibat
matibat approved these changes Nov 7, 2023
View reviewed changes
Copy link
Contributor

@matibat matibat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@thiagocarvalhodev thiagocarvalhodev merged commit 1a8585e into master Nov 7, 2023
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matibat matibat matibat approved these changes

Assignees

@thiagocarvalhodev thiagocarvalhodev

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@thiagocarvalhodev @matibat