Release: merge development into beta#90
Open
github-actions[bot] wants to merge 29 commits intobetafrom
Open
Conversation
…conflicts Downgrade devDependencies to match @nextcloud/eslint-config@8.4.x peer requirements (@vue/eslint-config-typescript ^13, eslint-plugin-vue ^9, eslint-plugin-n ^16, eslint-plugin-promise ^6, eslint-plugin-jsdoc ^46). Add missing production dependencies (@nextcloud/auth, vue-apexcharts). Fix eslint errors (html-indent, duplicate imports). Add license override for apexcharts (MIT-licensed but reports custom SPDX string).
…crash" This reverts commit 5fbb6e4.
fix: Resolve all CI quality check failures
Contributor
Author
Quality Report
Summary
PHP Quality
Vue Quality
Security
License Compliance
composer dependencies (94 total)
npm dependencies (248 total)
PHPUnit TestsPHPUnit tests were not enabled for this run. Integration Tests (Newman)Newman integration tests were not enabled for this run. Generated automatically by the Quality workflow.
|
Contributor
Author
Quality Report
Summary
PHP Quality
Vue Quality
Security
License Compliance
composer dependencies (94 total)
npm dependencies (248 total)
PHPUnit TestsPHPUnit tests were not enabled for this run. Integration Tests (Newman)Newman integration tests were not enabled for this run. Generated automatically by the Quality workflow.
|
Contributor
Author
Quality Report
Summary
PHP Quality
Vue Quality
Security
License Compliance
composer dependencies (94 total)
npm dependencies (248 total)
PHPUnit TestsPHPUnit tests were not enabled for this run. Integration Tests (Newman)Newman integration tests were not enabled for this run. Generated automatically by the Quality workflow.
|
Contributor
Author
Quality Report
Summary
PHP Quality
Vue Quality
Security
License Compliance
composer dependencies (94 total)
npm dependencies (248 total)
PHPUnit TestsPHPUnit tests were not enabled for this run. Integration Tests (Newman)Newman integration tests were not enabled for this run. Generated automatically by the Quality workflow.
|
Contributor
Author
Quality Report
Summary
PHP Quality
Vue Quality
Security
License Compliance
composer dependencies (94 total)
npm dependencies (248 total)
PHPUnit TestsPHPUnit tests were not enabled for this run. Integration Tests (Newman)Newman integration tests were not enabled for this run. Generated automatically by the Quality workflow.
|
Contributor
Author
Quality Report
Summary
PHP Quality
Vue Quality
Security
License Compliance
composer dependencies (94 total)
npm dependencies (248 total)
PHPUnit TestsPHPUnit tests were not enabled for this run. Integration Tests (Newman)Newman integration tests were not enabled for this run. Generated automatically by the Quality workflow.
|
The compiled JS contained a hard gate that showed 'OpenRegister is required' and blocked all UI routes. Replaced with a non-blocking NcNoteCard warning banner that allows the app to render normally while informing users about the missing configuration.
All specs moved from openspec/specs/ to openspec/changes/ with proper proposal.md as the entry point. Follows the spec-driven workflow: proposal -> design -> specs -> tasks
Add ADR references to the rules section of openspec/config.yaml so that ADR constraints are injected into openspec instructions output during artifact creation.
- 9 new SettingsServiceTest tests (CONFIG_KEYS, get/update, delegation) - Fixed SettingsControllerTest constructor (all 6 deps) - Feature doc with browser screenshot - Merged JS build fix for settings page rendering - Removed deprecated template entity type from spec
Full pipeline complete: proposal → design → specs → tasks → apply → verify → archive - 16/16 tasks completed - 9 unit tests added (SettingsServiceTest) - Feature doc with browser screenshot - Delta spec synced to openspec/specs/admin-settings/ as implemented baseline
Process all active OpenSpec changes through the full pipeline: Changes processed: - character-management: Character CRUD, stat calculation engine - dashboard: KPI cards, recent lists, skill chart widget layout - events-players: Event/player CRUD, event effect application - game-mechanics: Abilities, Effects, Skills, Items, Conditions - larping-skill-widget: GraphQL faceting donut chart - object-service: RegisterObjectFetcher data access layer - pdf-export: DocuDesk PDF character sheet export - rpg-system: Stat calculation pipeline with audit trail - search-service: Dead code documentation (removed) - register-config-json: Auto-import schemas on boot - user-settings: Settings dialog with re-import - deep-link-registration: Unified search URL patterns Implementation: - Created DeepLinkRegistrationListener for 8 object types - Registered listener in Application.php - Fixed named parameter bug in CharactersController cross-app calls Tests added (60 total, 152 assertions): - CharacterServiceTest (12 tests) - RegisterObjectFetcherTest (6 tests) - CharactersControllerTest (5 tests) - DashboardControllerTest (4 tests) - ConfigFileLoaderServiceTest (5 tests) - SettingsMapBuilderTest (8 tests) - DeepLinkRegistrationListenerTest (4 tests) Documentation: 12 feature docs + 6 screenshots created Specs: All 12 synced to openspec/specs/ and archived
Update documentation link from non-existent larpingapp.com to valid Gitbook URL.
Make ADR-011 more specific: list concrete directories to search and common duplications to prevent future utility reimplementations.
Unit tests MUST run inside the Docker container, not locally. Prevents false test failure reports from missing Nextcloud framework.
Adds a features overview document for the LARP campaign management app, covering character management, RPG game mechanics, events/players, PDF export, OpenRegister integration, and dashboard widget.
[Docs] Feature overview with GEMMA/TEC standards
Contributor
Author
Quality Report
Summary
PHP Quality
Vue Quality
Security
License Compliance
composer dependencies (94 total)
npm dependencies (248 total)
PHPUnit TestsPHPUnit tests were not enabled for this run. Integration Tests (Newman)Newman integration tests were not enabled for this run. Generated automatically by the Quality workflow.
|
Contributor
Author
Quality Report
Summary
PHP Quality
Vue Quality
Security
License Compliance
composer dependencies (94 total)
npm dependencies (248 total)
PHPUnit TestsPHPUnit 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.
|
Contributor
Author
Quality Report
Summary
PHP Quality
Vue Quality
Security
License Compliance
composer dependencies (94 total)
npm dependencies (248 total)
PHPUnit TestsPHPUnit 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.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated PR to sync development changes to beta for beta release.
Merging this PR will trigger the beta release workflow.