Skip to content

Added import.ps1#28

Open
rscholtelubberink wants to merge 5 commits into
mainfrom
added-import-functionality
Open

Added import.ps1#28
rscholtelubberink wants to merge 5 commits into
mainfrom
added-import-functionality

Conversation

@rscholtelubberink

Copy link
Copy Markdown
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

@Rick-Jongbloed Rick-Jongbloed left a comment

Copy link
Copy Markdown
Member

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

rhouthuijzen commented Sep 26, 2025

Copy link
Copy Markdown
Contributor

On-Hold

@JeroenBL

JeroenBL commented Oct 7, 2025

Copy link
Copy Markdown

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
@rhouthuijzen rhouthuijzen added the on-hold Work is temporarily paused, pending additional information, decisions, or capacity. label Jan 30, 2026
@mverbraak-t4e

Copy link
Copy Markdown

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

@JeroenBL dit gebeurt hier toch niet? Ik zit hier met @rschouten97 en we begrijpen de comment niet.

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 on-hold Work is temporarily paused, pending additional information, decisions, or capacity.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AcocuntReference of import differs from accountreference used in user lifecycle scripts

5 participants