Skip to content

Conversation

@artonge
Copy link
Collaborator

@artonge artonge commented Sep 10, 2025

  • Update dependencies for Vue3
  • Run npm lint:fix and fix reported warnings
  • Update usage of vue, vue-router and vuex
  • Remove unused isMobile import
  • Get rid of vue-virtual-grid Replaced with the internal virtual scrolling component.
  • Modernize router instantiation
  • Do not use Vue method to update objects
  • Use vue provided nextTick function
  • Mark semaphore object as raw
  • Migrate to Vue3 lib of leaflet

TODO

  • Migrate UploadPicker@nc/vue

@artonge artonge self-assigned this Sep 10, 2025
@artonge artonge force-pushed the artonge/chore/migrate_to_vue3 branch from 564186d to c6735eb Compare September 10, 2025 14:03
@codecov
Copy link

codecov bot commented Sep 10, 2025

Codecov Report

❌ Patch coverage is 1.86275% with 1001 lines in your changes missing coverage. Please review.
✅ Project coverage is 1.83%. Comparing base (b3d699c) to head (ad6f7f2).

Files with missing lines Patch % Lines
src/views/AlbumContent.vue 0.00% 126 Missing ⚠️
src/views/SharedAlbumContent.vue 0.00% 86 Missing ⚠️
src/components/GridLayout/GridLayout.vue 1.38% 71 Missing ⚠️
src/views/TimelineView.vue 0.00% 64 Missing ⚠️
src/services/downloadFiles.ts 1.92% 51 Missing ⚠️
src/components/FilesListViewer.vue 0.00% 46 Missing ⚠️
src/views/FoldersView.vue 0.00% 46 Missing ⚠️
src/views/SharedAlbums.vue 0.00% 41 Missing ⚠️
src/router/index.ts 0.00% 40 Missing ⚠️
...c/components/Albums/CollaboratorsSelectionForm.vue 0.00% 37 Missing ⚠️
... and 48 more
Additional details and impacted files
@@            Coverage Diff            @@
##           master   #3174      +/-   ##
=========================================
- Coverage    2.23%   1.83%   -0.41%     
=========================================
  Files         107     105       -2     
  Lines        6519    8500    +1981     
  Branches      121     123       +2     
=========================================
+ Hits          146     156      +10     
- Misses       6368    8338    +1970     
- Partials        5       6       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@artonge artonge changed the title refactor: Migrate to vue3 lib of leaflet refactor: Migrate to vue3 Sep 15, 2025
@artonge artonge changed the title refactor: Migrate to vue3 chore: Migrate to vue3 Sep 15, 2025
@artonge artonge force-pushed the artonge/chore/migrate_to_vue3 branch 2 times, most recently from 028ebdb to 73197d6 Compare September 17, 2025 16:26
@artonge artonge marked this pull request as ready for review September 17, 2025 16:34
@artonge artonge force-pushed the artonge/chore/migrate_to_vue3 branch 3 times, most recently from a447c9a to 9be1c3e Compare September 18, 2025 13:07
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
@artonge artonge force-pushed the artonge/chore/migrate_to_vue3 branch 2 times, most recently from 9f00645 to e85871b Compare September 18, 2025 13:26
It is not compatible with vue3.
We can replace it with the internal `VirtualScrolling` component as well as a custom grid layout component.

Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Prevent vue from converting the Semaphore object to a proxy as this is breaking private property access.

Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Because NcActions wants direct NcAction* children

Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Because NcActions wants direct NcAction* children.
And ajax endpoint is deprecated.

Signed-off-by: Louis Chemineau <louis@chmn.me>
And distinguish groups from users.

Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
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