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

Bringing back abstracted file sharing into kDrive #1006

Merged
merged 26 commits into from
Aug 10, 2023

Conversation

adrien-coye
Copy link
Contributor

@adrien-coye adrien-coye commented Jul 19, 2023

Abstract

Bring back the Core work related to file sharing in this app.

Also abstracted the on the fly Heic -> Jpg here

use date as default name Core
use date as default name CoreUI

Shared file name generation between manual import and PhotoLibraryUploader.

Edited pictures in Photo.app have now a non coliding name.

Details

  • Speedup improvement, scale import workers dynamicaly given a device.
  • chore: Bump Core / Core UI

Copy link
Contributor Author

@adrien-coye adrien-coye left a comment

Choose a reason for hiding this comment

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

I did a quick self review.

kDriveCore/Utils/Files/FileImportHelper.swift Outdated Show resolved Hide resolved
kDriveCore/Utils/Files/FileImportHelper.swift Outdated Show resolved Hide resolved
kDriveCore/Utils/Files/FileImportHelper.swift Outdated Show resolved Hide resolved
@adrien-coye adrien-coye marked this pull request as ready for review July 26, 2023 12:14
@adrien-coye adrien-coye requested a review from a team as a code owner July 26, 2023 12:14
@sonarcloud
Copy link

sonarcloud bot commented Aug 10, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@PhilippeWeidmann PhilippeWeidmann merged commit 159ecf2 into master Aug 10, 2023
3 checks passed
@PhilippeWeidmann PhilippeWeidmann deleted the shareExtensionAbstractCode branch August 10, 2023 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants