forked from spatie/laravel-medialibrary
-
Notifications
You must be signed in to change notification settings - Fork 2
Compatible with Laravel 10 and Laravel MongoDB #2
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
Open
ashen-1-dev
wants to merge
184
commits into
HorizonStack:main
Choose a base branch
from
WirStaff:sync-with-source-branch
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…patie#2925) * Fix clean command with custom path generators * Fix clean command to not remove original command Co-authored-by: tcostantini <[email protected]>
When implementing the `registerMediaConversions()` method, it is not clear that additional models need to be imported. Hopefully this PR makes it a little more clear for users with an IDE that does not automatically import the required models.
* add attachable support * Fix styling * fix phpstan errors * remove static analysis as it keeps failing only on github Co-authored-by: freekmurze <[email protected]>
Co-authored-by: Simon Verwaard <[email protected]>
…ection (spatie#2972) `$yourModel->clearMediaCollection()` doesn't delete all media in all collections. Is there any shorthand method to delete all media in all collections?
spatie#2993) * Added information that media wont be deleted if the deleted model uses soft deletes * Update retrieving-media.md Co-authored-by: Freek Van der Herten <[email protected]>
* Order media relationship * add+fix tests
* Adding a short vite description Thanks to @plakhin in spatie#2971 (comment) for the answer. I could only test the vue 3 change. * Added CSS Import for SPA use
Add a note about limitation of generators to deter users from going down the path of attempting to use this functionality for anything other than images.
) * Update CleanCommand to delete depreciated responsive images When running `php artisan media-library:clean` we should be clearing responsive images for conversions that are no longer used, as well as for active conversions that are no longer using responsive images. Fixes spatie#2397 * Add unit test for active conversions without responsive images * Remove duplicate test and file
It is better to use the `static` keyword instead of `self` in order to let the methods of this class beeing override when the class is extended see discussion here spatie#3020
This change corrects the documentation where the word "response" was used, but "request" was meant.
Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v3...v4) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Support drivers containing gcs or s3 in driver name
…patie#3395) The package vue-i18n (https://vue-i18n.intlify.dev/) offers a $tm-function to pass all keys at once as an array from a translation file. Maybe it's good to adapt this in the documentation. https://vue-i18n.intlify.dev/api/composition.html#tm-key
Bumps [stefanzweifel/git-auto-commit-action](https://github.com/stefanzweifel/git-auto-commit-action) from 4 to 5. - [Release notes](https://github.com/stefanzweifel/git-auto-commit-action/releases) - [Changelog](https://github.com/stefanzweifel/git-auto-commit-action/blob/master/CHANGELOG.md) - [Commits](stefanzweifel/git-auto-commit-action@v4...v5) --- updated-dependencies: - dependency-name: stefanzweifel/git-auto-commit-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
This commit removes the directory creation for the 'getMediaLibrary' function. According to Flysystem, the actual driver creates the required directory when attempting to write a file. > When writing files, the directory you’re writing to will be created automatically if and when that is required in the filesystem you’re writing to. If your filesystem does not require directories to exist (like AWS S3), the directory is not created. This is a performance consideration. Of course, you can always create the directory yourself by using the createDirectory operation.
* Delete orphaned media in clean command * Add 'getOrphans' to the media repository * Make delete orphaned opt-in, add test * Add collection name option, fix phpstan, add extra test * Fix tests * Fix test for L9 * Typo * Respect rate limit in remove orphaned media item method
* Don't include soft deleted models as orphaned media * In case a model is restored the media needs to stay around * extract orphansQuery method
I would like to suggest installing svgo through snap as an alternative to installing npm
Fix grammatically.
This reverts commit 27a16b6.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR provides sync with main source branch to make compatible with Laravel v10 and also change namespaces from Jenssegers\Mongodb.. to MongoDB\Laravel.. (commit) see https://github.com/mongodb/laravel-mongodbSync with source branch