Skip to content

feat(applications): page listing all applications - #57

Open
rubenvdlinde wants to merge 1 commit into
stack/s5-application-technical-namesfrom
stack/s6-applications-page
Open

rubenvdlinde wants to merge 1 commit into
stack/s5-application-technical-namesfrom
stack/s6-applications-page

Conversation

@rubenvdlinde

Copy link
Copy Markdown
Collaborator

Rebuild of #44 (originally #30) on the page model of nextcloud#3100, see #13. Stacked on #56.

What changes

An "All applications" entry in the Tables navigation opens one page with every application the user can open.

  • Applications page at /applications: a card per application, sorted by name, with its icon, description, number of menu items and technical name. A card opens the application's overview.
  • Create: an application can be created from this page as well.
  • Navigation: "All applications" sits at the top of the applications section.

Carried over from #44, and what changed because of nextcloud#3100

Carried over as is. Only the card link changed: the overview has its own route since #53 (/application/{id}/overview), because /application/{id} stays the start page per nextcloud#3099. The spec opens the Tables navigation before using the entry, since opening a new application still collapses it (unchanged behaviour, see #53).

Tests

Nextcloud 34 container (PostgreSQL 16), Playwright at 1600 x 1000, --timeout=120000:

  • application.spec.ts with the new "All applications are listed on one page that opens each of them" (the new application's card shows its 1 menu item, the dashboard, and opens its overview): 15 passed (1.7m).
  • Regression of the whole stack on this branch: context-grid-pages.spec.ts, context-menu.spec.ts, context.spec.ts, context-navigation.spec.ts, column-relation.spec.ts, tables-import-export-scheme.spec.ts, navigation-load-order.spec.ts: 22 passed (5.2m).
  • env -u ELECTRON_RUN_AS_NODE npx cypress run --component (all 14 specs): 81 of 82 passed; LinkCellPreview.cy.js failed once on "Failed to fetch dynamically imported module" from the Vite dev server and passed on its own (All specs passed! 9 9).
  • Whole unit suite on this branch: OK (561 tests, 1181 assertions).
  • npm run lint: 0 errors, 1 inherited warning (src/store/store.js:176); npm run stylelint: clean; npm run build: OK.

Prepared in the Conduction fork for porting upstream by @remko48. Prepared with Claude Code.

An "All applications" entry in the Tables navigation opens /applications: a
card per application, sorted by name, with its icon, description, number of
menu items and technical name. A card opens the application's overview, and
an application can be created from the page.

Assisted-by: ClaudeCode:claude-opus-5-5
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