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

Upgrade to v8 #1728

Merged
merged 121 commits into from
Mar 9, 2020
Merged

Upgrade to v8 #1728

merged 121 commits into from
Mar 9, 2020

Conversation

freekmurze
Copy link
Member

No description provided.

Copy link
Member

@AlexVanderbist AlexVanderbist left a comment

Choose a reason for hiding this comment

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

Heavy PR but great update 👍

Couple of nitpicks added in review & a couple notes right here:

  • media-library:* commands namechange missing in upgrading.md
  • media-library.php config file namechange missing in upgrading.md
  • Check composer dev dependencies (some are dependencies, others should be suggests)

docs/introduction.md Outdated Show resolved Hide resolved
src/Conversions/Conversion.php Outdated Show resolved Hide resolved
src/Conversions/ConversionCollection.php Outdated Show resolved Hide resolved
src/Conversions/ConversionCollection.php Outdated Show resolved Hide resolved
src/MediaCollections/FileAdder.php Outdated Show resolved Hide resolved
src/MediaCollections/Filesystem.php Outdated Show resolved Hide resolved
src/MediaCollections/Filesystem.php Outdated Show resolved Hide resolved
src/MediaCollections/HtmlableMedia.php Show resolved Hide resolved
src/MediaCollections/HtmlableMedia.php Show resolved Hide resolved
@freekmurze
Copy link
Member Author

Great code review, thanks @AlexVanderbist 👍

freekmurze and others added 5 commits March 9, 2020 17:14
* Use native file copy

If a file and the media destination are on the same disk, use native copy operation instead of streaming

* Fix disk
@freekmurze freekmurze merged commit 1f4dabf into master Mar 9, 2020
@freekmurze freekmurze deleted the v8 branch March 9, 2020 17:01
ipontt added a commit to ipontt/laravel-medialibrary that referenced this pull request Jun 19, 2021
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.

6 participants