✏️ De naam van de afzender komt uit de berichtenlijst (alsnog naar main) - #163
Merged
Merged
Conversation
De berichtenuitvraag draagt sinds MinBZK/moza-poc-fbs-berichtenbox#285 per bericht een `afzenderNaam`: verplicht, nooit leeg, en de weergavenaam van de organisatie in plaats van haar twintigcijferige nummer. Het veld `afzender`, dat hetzelfde nummer droeg als `magazijnId`, is daarmee verdwenen. Daarmee vervalt de reden waarom elke berichtenbox-pagina de namen van de organisaties per zitting in `sessionStorage` bewaarde. Dat mechanisme is weg — een sleutel, twee functies en hun uitzonderingen. Wat ervoor terugkomt is één vraag aan het stelsel: eerst de lijst, en alleen een ophaalronde als daar een 409 op komt. Dat is dezelfde weg als eerst, maar met het stelsel als bron van waarheid in plaats van een aantekening. Een aantekening hoort bij één tabblad, de sessie bij de ontvanger: in een tweede tabblad draaide er alsnog een hele ronde langs alle magazijnen terwijl de berichten klaarstonden. De prijs is één verzoek bij de eerste pagina van een zitting. Het organisatiefilter boven de lijst komt nu uit de lijst zelf, aangevuld met de organisaties die in een eigen ronde antwoordden maar niets te leveren hadden. Dat dicht een gat: een organisatie die haar bericht aanleverde buiten de ronde om stond niet in het filter, terwijl haar bericht wél in de lijst stond — filteren liet haar post dan verdwijnen zonder dat er iets te kiezen viel. De terugval op `magazijnId` blijft staan voor een uitvraag van vóór die wijziging. Onleesbaar voor een schermlezer, maar waar; en het schuift op zodra de omgevingen bij zijn. Het harnas van de keten-tests laat de lijst nu pas antwoorden nadat de ronde gedraaid heeft, zoals het stelsel dat doet — daarvóór 409. Zonder dat onderscheid zou een test die een ronde toetst er nooit een zien, nu de client met de lijst begint. Claude-Session: https://claude.ai/code/session_01233viQBxE5JqMV1ohZamRB (cherry picked from commit 1596f2a) Opnieuw toegepast op main. #153 had als basis de branch van #152 en werd tien seconden na #152 gemerged, in die basisbranch in plaats van in main. Daarmee kwam de wijziging nooit op main. Het conflict zat in het pollen uit #151, dat na deze wijziging op main kwam. Dat pollen gebruikte de namen uit de ronde (`organisatiesVanRonde`) om binnengekomen berichten een naam te geven. Die naam komt nu uit de lijst. Het pollen bouwt het organisatiefilter daarom op dezelfde manier op als het laden, met `magazijnenVan`. Een organisatie die buiten de ronde om post aanlevert, staat daarna dus ook in het filter. De polltests beginnen nu met een 409, zodat het laden nog een ronde draait. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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
2 participants
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.
#153 staat in GitHub als gemerged, maar de wijziging is nooit op
maingekomen. De PR was gestapeld op de branch van #152 (feat/keten-alle-berichten-ophalen). #152 ging om 15:06:59 naarmain, en tien seconden later werd #153 gemerged in die basisbranch in plaats van inmain. Daarna is die branch gewist. De merge-commit 87aaeba staat niet inmain, enafzenderNaamkomt opmainniet voor.Deze PR zet commit 1596f2a alsnog op
main. Wat die doet, staat in #153. Kort samengevat: de naam van de afzender komt uitafzenderNaamin de berichtenlijst, en de namen insessionStoragevervallen. Elke pagina vraagt eerst de lijst op en draait alleen een ronde als het stelsel met een 409 antwoordt. Het organisatiefilter komt uit de lijst zelf.Het conflict met #151
Het pollen uit #151 kwam na #153 op
mainen gebruikte de namen uit de ronde (organisatiesVanRonde) voor berichten die later binnenkomen. Wat er veranderd is:afzenderNaam. De beperking uit ➕ Nieuwe berichten verschijnen vanzelf in een openstaande berichtenbox #151, waarbij een organisatie die niet in de ronde zat als nummer verscheen, vervalt daarmee.magazijnenVan). Een organisatie die buiten de ronde om post aanlevert, staat daarna ook in het filter. Organisaties uit de ronde die niets te leveren hadden, blijven erin staan.organisatiesVanRondeblijft bestaan, maar alleen voor die laatste groep.Tests
npm test: 468 geslaagd, 1 mislukt. Die ene isdetail.test.js› "noemt de knop Terugzetten in inbox als het bericht in de prullenbak staat". Die test faalt opmainook al, en hangt niet samen met deze wijziging.Deze PR is de basis voor de PR die de berichtenbox op
_volgenaansluit (MinBZK/MijnOverheidZakelijk#939).🤖 Generated with Claude Code