Skip to content

Conversation

@rscholtelubberink
Copy link
Contributor

First version importscripts

@JeroenBL JeroenBL added the needs-review Indicates that the PR is ready and waiting for review label Jul 18, 2025
@rscholtelubberink rscholtelubberink added the current-sprint PRs to be addressed within the current sprint label Jul 25, 2025
Copy link
Member

@Rick-Jongbloed Rick-Jongbloed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed, please update the mapping logic so that Insite and Outsite are correctly mapped to their corresponding fields. This may also apply to additional fields if relevant.

@rhouthuijzen rhouthuijzen removed the current-sprint PRs to be addressed within the current sprint label Sep 26, 2025
@rhouthuijzen
Copy link
Contributor

rhouthuijzen commented Sep 26, 2025

On-Hold

@JeroenBL
Copy link

JeroenBL commented Oct 7, 2025

In de huidige implementatie worden gebruikers eerst in meerdere pagina’s opgehaald met een do/while-lus en vervolgens in een aparte foreach weggeschreven naar HelloID. Dit resulteert in dubbele iteraties en onnodig geheugenverbruik.

Het doel is om beide stappen te combineren: tijdens het ophalen van elke pagina direct de mapping uitvoeren en het resultaat outputten, zonder tussenopslag in $importedAccounts.

Huidige situatie

  • do/while-lus haalt gebruikerspagina’s op en voegt deze toe aan $importedAccounts.
  • foreach-lus doorloopt daarna alle $importedAccounts en schrijft resultaat naar HelloID

Gewenste situatie

  • Tijdens het ophalen van elke pagina (binnen de do/while-lus) direct en terugschrijven naar HelloID.
  • $importedAccounts vervalt volledig.

Voordelen

  • Minder geheugenverbruik
  • Minder code en eenvoudiger leesbaarheid
  • Betere performance bij grote datasets

Actiepunten

  1. Verwijder $importedAccounts-variabele
  2. Verplaats mapping-logica binnen de do/while-lus
  3. Test of alle records correct worden verwerkt bij meerdere pagina’s

Added insite/outsite
Removed importedAccounts variable and used direct streaming
Fixes after testing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review Indicates that the PR is ready and waiting for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants