chore: update generated files from PR #11484#41
Conversation
There was a problem hiding this comment.
Static review only, per the environment constraints. I found one issue: the generated types expose country-aware Headless API filtering, but the package's exported convenience client still has no way to send country, so consumers using createHeadlessApiClient cannot actually use the new behavior.
| language?: unknown; | ||
| language?: string; | ||
| /** @description ISO 3166-1 alpha-2 country code for the customer. Used to filter rewards by country availability. If not provided, the site default is used */ | ||
| country?: string; |
There was a problem hiding this comment.
This new country parameter needs a matching change in the exported client wrapper. createHeadlessApiClient currently accepts global channel and language, then sends them as headers for every call, but it has no country option and none of the wrapper methods include params.query. Since this parameter is what filters configuration/customer/reward responses by country availability, consumers using the main client API will still always get the site default country even after these generated types are published. Please add a country?: string option or per-call country support in src/client.ts and thread it through the generated operations that now accept this query param.
Auto-generated by the CI pipeline.
Source: https://github.com/loyaltylion/hogwarts/pull/11484
These files were regenerated from changes to the API contracts. Review and merge alongside the main PR.
Generated files:
src/types/api.tssrc/types/schemas.ts