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

feat(Files): Preview videos files in Storage and Chats #1699

Merged
merged 86 commits into from
Jan 23, 2024

Conversation

lgmarchi
Copy link
Contributor

@lgmarchi lgmarchi commented Jan 11, 2024

What this PR does 📖

1. Preview videos

a. If is first time, and video is download with success: WIll be show a loader, and after that the video player will start with video

Screen.Recording.2024-01-15.at.10.05.18.mov

b. If is not first time previewing that file during app execution, it will load quick from cache and open video player with video without showing the loader

Screen.Recording.2024-01-15.at.10.07.21.mov

c. If take more than 10 seconds to download and load video, it will return a video player with error

Screen.Recording.2024-01-15.at.10.09.07.mov

2. Preview Image

a. If is first time, and image is downloaded with success: WIll be show a loader, and after that will open image in maximum quality

Screen.Recording.2024-01-15.at.11.13.58.mov

b. If is not first time previewing that image during app execution, it will load quick from cache and open image with maximum quality

c. If take more than 1.5 seconds to download and load image, it will return thumbnail of the image with lower quality

Screen.Recording.2024-01-15.at.11.14.59.mov

3. Cache

a. It will clear cache everytime app is closed or opened.

Screen.Recording.2024-01-15.at.10.11.00.mov

Which issue(s) this PR fixes 🔨

Special notes for reviewers 🗒️

Additional comments 🎤

@github-actions github-actions bot added Don't merge yet DO NOT MERGE Missing dev review Still needs to be reviewed by a dev labels Jan 11, 2024
Copy link
Contributor

github-actions bot commented Jan 11, 2024

UI Automated Test Results Summary for MacOS/Windows

552 tests   500 ✅  3h 38m 52s ⏱️
 43 suites   52 💤
  3 files      0 ❌

Results for commit 50c88f5.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Jan 11, 2024

UI Automated Tests execution is complete! You can find the test results report here

@luisecm luisecm added the linter failing Cargo Workflow (linter) failed on this PR label Jan 11, 2024
@stavares843
Copy link
Member

all the above was in macOS, m1

@github-actions github-actions bot added the missing fixing conflict A conflict exists in current PR and needs resolution label Jan 22, 2024
@github-actions github-actions bot added linter failing Cargo Workflow (linter) failed on this PR Failed Automated Test This PR is failing Luis's Appium test and needs revised labels Jan 22, 2024
Copy link
Member

@stavares843 stavares843 left a comment

Choose a reason for hiding this comment

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

Captura de ecrã 2024-01-22, às 20 20 06

does not compile

macOS, m1

@stavares843 stavares843 added Changes requested When other dev or QA request a change and removed Ready for testing Ready for QA to test labels Jan 22, 2024
@lgmarchi lgmarchi added Ready for testing Ready for QA to test and removed Changes requested When other dev or QA request a change missing fixing conflict A conflict exists in current PR and needs resolution linter failing Cargo Workflow (linter) failed on this PR labels Jan 23, 2024
@github-actions github-actions bot removed the Failed Automated Test This PR is failing Luis's Appium test and needs revised label Jan 23, 2024
@stavares843 stavares843 added QA Tested QA has tested and approved and removed Ready for testing Ready for QA to test labels Jan 23, 2024
@dariusc93 dariusc93 added Ready to Merge This PR is ready to merge and removed Missing dev review Still needs to be reviewed by a dev Don't merge yet DO NOT MERGE labels Jan 23, 2024
@stavares843 stavares843 merged commit 03cd078 into dev Jan 23, 2024
5 checks passed
@stavares843 stavares843 deleted the preview-videos-files branch January 23, 2024 20:55
@github-actions github-actions bot removed QA Tested QA has tested and approved Ready to Merge This PR is ready to merge labels Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants