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

adaptation for names and replace content #64

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

wilzamguerrero
Copy link

Hi
1 Purpose: This adaptation was made to prevent a suffix from being added to the filenames. The original add_uuid_to_filename function added a timestamp-based suffix to the filenames. To avoid this undesired modification to the names, we changed the function so that it simply returns the filename as provided, without altering its original format.

2 Purpose: The copy_if_different function was created to improve the file copying process in the api_add_to_collections function. The goal is to avoid overwriting files that already exist in the destination folder if they are identical to the source files. This function only copies a file if it doesn't exist in the destination folder or if it is different (by comparing its content). This way, the copying process is optimized, avoiding unnecessary duplicates and overwrites.

Additionally, this logic was integrated into the api_add_to_collections function to handle both individual files and complete directories.

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.

1 participant