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

[Woo POS] Add tracking when pagination response headers are missing #14737

Open
jaclync opened this issue Dec 20, 2024 · 1 comment
Open

[Woo POS] Add tracking when pagination response headers are missing #14737

jaclync opened this issue Dec 20, 2024 · 1 comment
Labels
feature: POS priority: low Not many people are affected or there’s a workaround, etc. type: task An internally driven task.

Comments

@jaclync
Copy link
Contributor

jaclync commented Dec 20, 2024

Ref: #14728 (comment)

In the discussion about whether to throw an error when response headers are missing for pagination support, we want to evaluate whether the fallback implementation is worthwhile. We will add tracking when the pagination response headers are missing from the POS products API request.

After a month or so with data tracking, if the percentage is almost zero (like < 1%), we can remove the fallback implementation and throw an error in case of missing pagination headers.

@jaclync jaclync added feature: POS priority: low Not many people are affected or there’s a workaround, etc. type: task An internally driven task. labels Dec 20, 2024
@dangermattic
Copy link
Collaborator

Thanks for reporting! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature: POS priority: low Not many people are affected or there’s a workaround, etc. type: task An internally driven task.
Projects
None yet
Development

No branches or pull requests

2 participants