Skip to content

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

Open
remko48 wants to merge 2 commits into
stack/d2-application-technical-namesfrom
stack/d3-applications-page
Open

remko48 wants to merge 2 commits into
stack/d2-application-technical-namesfrom
stack/d3-applications-page

Conversation

@remko48

@remko48 remko48 commented Oct 14, 2026

Copy link
Copy Markdown
Member

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

What changes

  • 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.
  • Tests: a Playwright test opens the page and an application from it.

🖼️ Screenshots

🏚️ Before 🏡 After
|

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔙 Backport requests are created or not needed: /backport to stableX.X
  • 📅 Milestone is set
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

- "All applications" in the navigation opens one page with every
  application, its description, menu size and technical name
- an application can be created from there as well

Signed-off-by: Remko <remko@conduction.nl>
Assisted-by: ClaudeCode:claude-opus-5-5
@remko48
remko48 added this pull request to stack #32 October 14, 2026 10:03
@remko48
remko48 removed this pull request from stack #32 October 14, 2026 10:21
@remko48
remko48 added this pull request to stack #33 October 14, 2026 10:21
@remko48 remko48 closed this Oct 14, 2026
@remko48 remko48 reopened this Oct 14, 2026
</p>
<p class="application-card__meta">
{{ n('tables', '%n menu item', '%n menu items', (context.menuItems ?? []).length) }}
<template v-if="context.technicalName">

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Minor — the card shows /technical_name for other people's applications

#29 addresses other people's applications by id, because technical names are only unique per owner and /apps/tables/app/{name} resolves to the viewer's own application first. This card still shows the name for a shared application, suggesting an address that may open a different app.

Impact: a misleading address on shared applications.

Suggested fix: v-if="context.technicalName && ownsContext(context)" (with permissionsMixin).

@rubenvdlinde

Copy link
Copy Markdown
Collaborator

Superseded by #44 (rebuilt on application pages, see #13).

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