Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mailchimp-API: fetch må støtte pagination når vi får for mange resultater #60

Open
mariusmandal opened this issue Oct 24, 2019 · 4 comments · May be fixed by #145
Open

Mailchimp-API: fetch må støtte pagination når vi får for mange resultater #60

mariusmandal opened this issue Oct 24, 2019 · 4 comments · May be fixed by #145
Assignees
Labels
👨🏼‍💻 forbedring Forbedring av eksisterende funksjon

Comments

@mariusmandal
Copy link
Member

Problemet nå:

  • databasen har 10 tags
  • mailchimp har 240 tags
  • pageSize var 50

Når en mottaker skal tagges med Viken (som tilfeldigvis hentes inn et sted mellom 51 og 240), feiler hele greia fordi a) vi finner ikke tag, b) vi prøver å opprette, c) Mailchimp sier den finnes, d) Vi har ingen måte å finne den på (fordi mailchimp ikke støtter søk etter tags via API). Bah.

Dvs at loadAll() automatisk må håndtere pagination, og kjøre på helt til alle er hentet inn.

Midlertidig er grensen satt til 1000, som berger oss noen tags videre

@mariusmandal
Copy link
Member Author

Tatt inn i feature-sys

@mariusmandal
Copy link
Member Author

Tatt inn i sprint 4

@mariusmandal mariusmandal reopened this May 10, 2021
@mariusmandal
Copy link
Member Author

Gjenskape problemet

Du vil ikke få opp feilen nå, ettersom vi enda ikke har brutt 1000-tag-grensa, men:

  • Logg inn som UKM Norge
  • Velg en kommune
  • Legg til en ny administrator

@mariusmandal
Copy link
Member Author

Løsningsforslag

Finn en måte som gjør at vi automatisk henter neste side med tags hvis det finnes flere enn vi får på første request.
Dessverre husker jeg ikke hvordan koden er skrevet, så her går det nok litt tid til research.

API-koden: https://github.com/UKMNorge/UKMapi/tree/master/API/Mailchimp

Her bruker vi koden:
https://github.com/UKMNorge/UKMnettverket/blob/master/controller/administrator-manage.controller.php#L113-L163

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👨🏼‍💻 forbedring Forbedring av eksisterende funksjon
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants