Media Gallery Vue 3
Work in progress.
The goal here is first to migrate all the components and SFC to Vue 3. We cannot use new Vue with Vue 3 so some components will need to be refactored.
Also, the components dynamic loading methods need to be removed in favor of using a wrapping component with props simply. Favor usage of Vite with Vue 3 apps/components.
Tasks
- [ ] Migrate Media Library to Vue 3
- [ ] Migrate MediaField Media Picker to Vue 3
- [ ] Make Media Library (standalone) by using a different modal than the Bootstrap one. https://vue-final-modal.org/
- [ ] Make Media Library (standalone) by using display Flex and Grid simply instead of using Bootstrap classes for styling.
- [ ] Check if it #15569 is fixed by this; if yes, close it.
- [ ] Check for other Media issues that might be fixed by this.
- [ ] Make thumbnail view optional
- [ ] Add a Swagger/OpenApi module for Media Gallery ApiController
- [ ] Normalize API requests responses. Make use of ProducesResponseType in Api controller.
- [ ] Add SignalR notifier. Which means we need to have a SignalR client implementation.
- [ ] Upgrade Azure Blob storage to latest
Fixes https://github.com/OrchardCMS/OrchardCore/issues/7110.
Optional
Consider refactoring upload bits by using Uppy instead of JQuery FileUpload as it is now archived and old.
See:
https://uppy.io/ https://tus.io/ https://github.com/tusdotnet/tusdotnet https://www.ietf.org/archive/id/draft-tus-httpbis-resumable-uploads-protocol-00.html
@piedone This one will be my only concern in the short term.
Gotcha!
This pull request has merge conflicts. Please resolve those before requesting a review.
This pull request has merge conflicts. Please resolve those before requesting a review.
This pull request has merge conflicts. Please resolve those before requesting a review.
This pull request has merge conflicts. Please resolve those before requesting a review.
This pull request has merge conflicts. Please resolve those before requesting a review.
This pull request has merge conflicts. Please resolve those before requesting a review.
Working in OC.
Working as standalone (using a baseHost)
Works fine here @hyzx86
<media-app
site-id="xtwa3swgv9favzbynjd81d8e5v"
base-host="https://localhost:5001"
base-path=""
upload-files-url="/api/media/upload"
:max-upload-chunk-size="0"
></media-app>
This pull request has merge conflicts. Please resolve those before requesting a review.
This pull request has merge conflicts. Please resolve those before requesting a review.
This pull request has merge conflicts. Please resolve those before requesting a review.
It seems that this pull request didn't really move for quite a while. Is this something you'd like to revisit any time soon or should we close? Please comment if you'd like to pick it up.
Of course bot I will pick it up at some time in the future 😄
This pull request has merge conflicts. Please resolve those before requesting a review.
for instructions on how to resolve the merge conflicts due to #16572 please follow the step listed in this comment.