Skip to content

Refactors data source page and component cards#934

Merged
rubenvdlinde merged 11 commits into
feature/nextcloud-vue-package-usagefrom
feature/REGISTERS-480/data-sources-page
Mar 22, 2026
Merged

Refactors data source page and component cards#934
rubenvdlinde merged 11 commits into
feature/nextcloud-vue-package-usagefrom
feature/REGISTERS-480/data-sources-page

Conversation

@SudoThijn
Copy link
Copy Markdown
Contributor

@SudoThijn SudoThijn commented Mar 18, 2026

Refactors the data sources management page and various component cards to leverage generic UI components from @conduction/nextcloud-vue.

  • Data Sources Page (SourcesIndex.vue): Migrates the page to use the CnIndexPage component, consolidating listing, pagination, filtering, and action handling. This includes utilizing the built-in form and delete dialogs, removing the need for custom EditSource and DeleteSource modals, and significantly reducing boilerplate code.
  • Component Cards (ApplicationCard.vue, OrganisationCard.vue): Replaces custom card implementations with the generic CnCard component, enhancing UI consistency and reducing maintenance by abstracting common card layouts.

This update improves code maintainability, reusability, and consistency across the application by adopting standardized UI components.

based off of: #932
requires: ConductionNL/nextcloud-vue#16

@SudoThijn SudoThijn changed the title refactors data sources page to make use of nextcloud-vue package Refactors data source page and component cards Mar 18, 2026
Base automatically changed from feature/REGISTERS-479/Applicaties-page to feature/nextcloud-vue-package-usage March 20, 2026 08:07
@rubenvdlinde rubenvdlinde merged commit 8a13cab into feature/nextcloud-vue-package-usage Mar 22, 2026
1 check passed
@rubenvdlinde rubenvdlinde deleted the feature/REGISTERS-480/data-sources-page branch March 22, 2026 21:14
@remko48 remko48 restored the feature/REGISTERS-480/data-sources-page branch March 30, 2026 07:20
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