Skip to content

fix: Proper NcAppSidebar on detail pages#994

Open
rubenvdlinde wants to merge 4 commits intodevelopmentfrom
fix/detail-page-proper-sidebar
Open

fix: Proper NcAppSidebar on detail pages#994
rubenvdlinde wants to merge 4 commits intodevelopmentfrom
fix/detail-page-proper-sidebar

Conversation

@rubenvdlinde
Copy link
Contributor

Summary

  • Add objectSidebarState provide/inject pattern to App.vue
  • Detail pages now render sidebar as proper NcAppSidebar (sibling of NcAppContent)

Depends on ConductionNL/nextcloud-vue#32
Closes #993

Search result subtitles previously showed raw numeric IDs like
"Schema: 498 • Register: 228". Now resolves to human-readable
titles like "Client • Pipelinq" using SchemaMapper and RegisterMapper
with a per-request name cache to avoid repeated lookups.

Closes #975
- tmlo-metadata: TMLO metadata standard support for archival compliance
- nextcloud-entity-relations: Nextcloud entity relations integration
@github-actions
Copy link
Contributor

Quality Report

Repository ConductionNL/openregister
Commit d897cce
Branch 994/merge
Event pull_request
Generated 2026-03-24 16:05 UTC
Workflow Run https://github.com/ConductionNL/openregister/actions/runs/23499370737

Summary

Group Result
PHP Quality FAIL
Vue Quality PASS
Security PASS
License PASS
PHPUnit SKIP
Newman SKIP
Playwright SKIP

PHP Quality

Tool Result
lint PASS
phpcs FAIL
phpmd PASS
psalm PASS
phpstan PASS
phpmetrics PASS

Vue Quality

Tool Result
eslint PASS
stylelint PASS

Security

Ecosystem Result
composer PASS
npm PASS

License Compliance

Ecosystem Result
composer PASS
npm PASS

composer dependencies (147 total)

Metric Count
Approved (allowlist) 146
Approved (override) 1
Denied 0

npm dependencies (586 total)

Metric Count
Approved (allowlist) 585
Approved (override) 1
Denied 0

PHPUnit Tests

PHPUnit tests were not enabled for this run.

Integration Tests (Newman)

Newman integration tests were not enabled for this run.

E2E Tests (Playwright)

Playwright E2E tests were not enabled for this run.


Generated automatically by the Quality workflow.

Download the full PDF report from the workflow artifacts.

@rubenvdlinde rubenvdlinde changed the base branch from main to development March 25, 2026 15:30
@github-actions
Copy link
Contributor

Quality Report

Repository ConductionNL/openregister
Commit 060f595
Branch 994/merge
Event pull_request
Generated 2026-03-25 15:39 UTC
Workflow Run https://github.com/ConductionNL/openregister/actions/runs/23549653779

Summary

Group Result
PHP Quality PASS
Vue Quality PASS
Security PASS
License PASS
PHPUnit SKIP
Newman SKIP
Playwright SKIP

PHP Quality

Tool Result
lint PASS
phpcs PASS
phpmd PASS
psalm PASS
phpstan PASS
phpmetrics PASS

Vue Quality

Tool Result
eslint PASS
stylelint PASS

Security

Ecosystem Result
composer PASS
npm PASS

License Compliance

Ecosystem Result
composer PASS
npm PASS

composer dependencies (147 total)

Metric Count
Approved (allowlist) 146
Approved (override) 1
Denied 0

npm dependencies (595 total)

Metric Count
Approved (allowlist) 592
Approved (override) 3
Denied 0

PHPUnit Tests

PHPUnit tests were not enabled for this run.

Integration Tests (Newman)

Newman integration tests were not enabled for this run.

E2E Tests (Playwright)

Playwright E2E tests were not enabled for this run.


Generated automatically by the Quality workflow.

Download the full PDF report from the workflow artifacts.

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.

fix: Add objectSidebarState for proper NcAppSidebar on detail pages

1 participant