Skip to content

chore: update to vuejs3 and nc34#1527

Open
TheoLallement wants to merge 10 commits intonextcloud:masterfrom
TheoLallement:feat/update-front-for-nc34
Open

chore: update to vuejs3 and nc34#1527
TheoLallement wants to merge 10 commits intonextcloud:masterfrom
TheoLallement:feat/update-front-for-nc34

Conversation

@TheoLallement
Copy link

@TheoLallement TheoLallement commented Mar 9, 2026

Hello,
First contribution for me so sorry in adance if i didn't do everything right.
There is a lot of non regression to do but I don't know all the features of the app so it's kinda hard to test this.

Main changes

Update of Vue.Js parts of code to be compatible with new Nextcloud versions.

  • migrating to new nextcloud api
  • migrating to vue3
  • remove use of library vue2-leaflet

Migrating to vue 3

  • replace all of this.$set(array, index, newValue) by this.array[key] = newValue
  • use createApp instead of Vue
  • remove vue-click-outside package and make custom directive
  • rename slot="customSlot" to #slot

Migrating to new nextcloud api (v9.x.x to have vue3 compat)

  • add a tooltip global directive (tooltip was deprecated in v8.x.x and deleted in v9.x.x which is needed for vue3)
  • stop using window.OCA.Files.Sidebar.setActiveTab and replaced it by a activeTab props
  • replaced generic import by named import : import { NcActionButton } from '@nextcloud/vue' instead of import NcActionButton from '@nextcloud/vue/dist/Components/NcActionButton.js'
  • replace isMobile by useIsMobile (deleted in v9.x.x)
  • put all the NcCounterBubble inside <template #counter> and use the :count props

Replace vue2-lealfet by leaflet

Hardest part to explain and document. WIP

Todo

in another pr maybe to not make this one bigger than it already is

  • migrate to vite
  • type some main functions

@TheoLallement TheoLallement changed the title Feat/update front for nc34 chore: update to vuejs3 and nc34 Mar 9, 2026
Signed-off-by: Théo Lallement <lallement.theo@gmail.com>
Signed-off-by: Théo Lallement <lallement.theo@gmail.com>
Signed-off-by: Théo Lallement <lallement.theo@gmail.com>
Signed-off-by: Théo Lallement <lallement.theo@gmail.com>
Signed-off-by: Théo Lallement <lallement.theo@gmail.com>
Signed-off-by: Théo Lallement <lallement.theo@gmail.com>
Signed-off-by: Théo Lallement <lallement.theo@gmail.com>
Signed-off-by: Théo Lallement <lallement.theo@gmail.com>
Signed-off-by: Théo Lallement <lallement.theo@gmail.com>
@TheoLallement TheoLallement force-pushed the feat/update-front-for-nc34 branch from ee7b2f6 to 1dd48da Compare March 9, 2026 18:46
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.

1 participant