Repository navigation
Samla användarverktyg under Profile och koppla filmer till hjälpartiklar - #273
Merged
Merged
Conversation
jmfk
force-pushed
the
codex/profile-instruction-films
branch
from
October 22, 2026 23:51
85e868a to
99ecba4
Compare
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
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.
Inbox, Kanban och felrapportering låg direkt i huvudmenyn och använde ett eget visuellt uttryck. Instruktionsfilmer kunde inte kopplas till hjälpartiklar eller laddas upp i den lokala MinIO-miljön. Den här batchen gör användarverktygen tillgängliga via Profile, anpassar dem till ECEEE:s befintliga UI och kopplar privata filmer till rätt dokumentation.
Ändringar
eceee-v4-design-skill med styrning i AGENTS.md.help_guide_id/helpGuideId, tenantavgränsat listfilter och ett tenant/guide-index. Avpublicerade filmer returneras inte för artikelkopplingen.lodashsom direkt produktionsberoende.Beroende
Provider-PR: https://github.com/jmfk/shared-local-infrastructure/pull/3
Providerändringen måste finnas innan
make configure-local-infrakörs med denna version. Den provisionerar bara ECEEE:s privata bucket och en begränsad kompletterande MinIO-policy. Andra pågående ändringar i infrastrukturrepot ingår inte.Lokal verifiering mot senaste main
npm ci --ignore-scripts --no-audit --no-fund— passerar.npm run lint— passerar, 0 fel; 969 varningar i den befintliga frontendkodbasen.npm run build— passerar; kvarvarande varningar om chunkstorlek/dynamiska importer.python manage.py test tenant_support --settings=tenant_support.test_settings_postgres --noinput --verbosity=1— 30 passerar mot en isolerad PostgreSQL 17-instans, inga överhoppade tester.python3 -m unittest discover -s scripts -p test_configure_orbstack.py— 14 passerar.makemigrations tenant_support --check --dry-runoch skillvalidering — passerar..env, nyckel, DB-dump eller lokal testdata ingår i PR:n.Migrering och avgränsningar
tenant_support.0002_instruction_help_guideär en additiv schemamigrering: artikelkopplingen är valfri och ett tenant/guide-index skapas. Kolumnen tillåter NULL så den gamla backendversionen kan fortsätta publicera mellan migrering och omstart; API och idempotenta återförsök normaliserar detta till en tom sträng. Ren installation täcks av isolerade tester. Ett regressionstest använder modellen från migrering 0001 för att verifiera publicering från den tidigare versionen och återförsök mot den nya versionen efter schemauppgraderingen. Inga videofiler skrivs om, inga notiser genereras och inga destruktiva dataoperationer ingår. Indexskapandet kan låsa tabellen under migrering; en produktionsuppgradering behöver därför ordinarie migrerings-/deploygranskning. Roll-forward föredras framför att ta bort kolumnen och förlora artikelkopplingar.Hela frontend-/backend-/Playwright-sviten har inte körts lokalt. PostgreSQL-specifika samtidighetstester har passerat lokalt; full integrationskörning för aktuell head återstår till CI/full-ci-gaten före eventuell merge. Batchen är en sammanhängande användarresa och motiverar en samlad CI-körning. Ingen merge eller produktionsdeploy ingår.