Skip to content

Minor fixes#14

Merged
NoResponseMate merged 6 commits intoSylius:mainfrom
mpysiak:import-poc
Jul 16, 2025
Merged

Minor fixes#14
NoResponseMate merged 6 commits intoSylius:mainfrom
mpysiak:import-poc

Conversation

@mpysiak
Copy link
Member

@mpysiak mpysiak commented Jul 15, 2025

No description provided.

@mpysiak mpysiak force-pushed the import-poc branch 3 times, most recently from 31a3dc7 to bc77ba8 Compare July 16, 2025 06:56
Comment on lines +27 to +36
if (!is_dir($this->importFilesDirectory)) {
mkdir($this->importFilesDirectory, 0755, true);
}

$fileName = uniqid() . '_' . $file->getClientOriginalName();
$filePath = $this->importFilesDirectory . '/' . $fileName;

$file->move($this->importFilesDirectory, $fileName);

return $filePath;
Copy link
Contributor

Choose a reason for hiding this comment

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

Would be nice to save the path on the process so it can be retried, or to perform cleanup after the import is done.

Copy link
Member Author

Choose a reason for hiding this comment

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

In next iteration

@mpysiak mpysiak force-pushed the import-poc branch 4 times, most recently from 978e387 to 4f82a57 Compare July 16, 2025 10:13
@NoResponseMate NoResponseMate merged commit 3b12bb5 into Sylius:main Jul 16, 2025
3 checks passed
@mpysiak mpysiak deleted the import-poc branch July 16, 2025 10:38
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.

2 participants