fix(git): de opslagmelding zegt wat de warnings echt betekenen

brenno commented

2026-07-23 17:28:18 +00:00

Owner

Aanleiding: een bevinding van de documentatieronde bij #756. De snackbar na een git-opslag met waarschuwingen zei "video en audio gaan (nog) niet mee naar git" — achterhaald sinds de git-opslagfasen: media wordt gewoon gepoold en gecommit (poolAsset over videoPath/audioPath in deck_repo_serializer.dart). De waarschuwingen waarop de melding afgaat betekenen "verwijzing niet leesbaar, zonder bruikbare extensie, of buiten het project" — en dat zegt de melding nu, in alle 31 talen. Het doc-commentaar op GitSaveResult.warnings droeg dezelfde verouderde claim en is meegecorrigeerd.

Regressietest, eerst rood gezien (test/git_save_snackbar_wording_test.dart): de nieuwe formulering bestaat in elke taal, én de oude sleutel is uit alle vertaaltabellen — die tweede helft bewaakt dat een merge=union-rebase van twee l10n-takken de verwijderde belofte niet stil laat herrijzen.

Geen kernwaarden-raakvlak (formaat, opslag, afhankelijkheden en uitgaand verkeer onaangeroerd; dit maakt een bestaande interfacebelofte juist wáár) — bewaker-stap daarom bewust overgeslagen.

Gedraaid: make check groen, check-secrets 0, sast 0.

🤖 Generated with Claude Code

Aanleiding: een bevinding van de documentatieronde bij #756. De snackbar na een git-opslag met waarschuwingen zei "video en audio gaan (nog) niet mee naar git" — achterhaald sinds de git-opslagfasen: media wordt gewoon gepoold en gecommit (`poolAsset` over `videoPath`/`audioPath` in deck_repo_serializer.dart). De waarschuwingen waarop de melding afgaat betekenen "verwijzing niet leesbaar, zonder bruikbare extensie, of buiten het project" — en dat zegt de melding nu, in alle 31 talen. Het doc-commentaar op `GitSaveResult.warnings` droeg dezelfde verouderde claim en is meegecorrigeerd. **Regressietest, eerst rood gezien** (`test/git_save_snackbar_wording_test.dart`): de nieuwe formulering bestaat in elke taal, én de oude sleutel is uit alle vertaaltabellen — die tweede helft bewaakt dat een merge=union-rebase van twee l10n-takken de verwijderde belofte niet stil laat herrijzen. Geen kernwaarden-raakvlak (formaat, opslag, afhankelijkheden en uitgaand verkeer onaangeroerd; dit maakt een bestaande interfacebelofte juist wáár) — bewaker-stap daarom bewust overgeslagen. **Gedraaid:** make check groen, check-secrets 0, sast 0. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

brenno added 2 commits

2026-07-23 17:28:18 +00:00

fix(git): de opslagmelding zegt wat de warnings echt betekenen (#756-restpunt) 8729c3dad7

'Video en audio gaan (nog) niet mee' was achterhaald: media wordt
gewoon gepoold en gecommit. De warnings zijn verwijzingen die niet
konden worden gelezen of buiten het project wijzen, en dat zegt de
melding nu. Regressietest eerst rood gezien; die bewaakt ook dat de
oude sleutel niet via een l10n-merge terugkeert.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

l10n: de nieuwe opslagmelding in alle 31 talen, de oude sleutel weg

ci / gate (pull_request) Failing after 29m20s

Details

2d328265e9

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

brenno merged commit b196c14f04 into main

2026-07-23 17:28:33 +00:00

brenno referenced this pull request from a commit

2026-07-23 17:28:34 +00:00

Merge pull request 'fix(git): de opslagmelding zegt wat de warnings echt betekenen' (#770) from fix/git-snackbar-media into main

Rows
Columns

fix(git): de opslagmelding zegt wat de warnings echt betekenen #770