Skip to content

Revert "Migrate to Storefront API 2026-07 + remove deprecated fields (v3.0.0)" - #146

Merged
imsujan276 merged 1 commit into
mainfrom
revert-143-main
Jul 15, 2026
Merged

Revert "Migrate to Storefront API 2026-07 + remove deprecated fields (v3.0.0)"#146
imsujan276 merged 1 commit into
mainfrom
revert-143-main

Conversation

@imsujan276

Copy link
Copy Markdown
Owner

Reverts #143

@imsujan276
imsujan276 merged commit e1c8744 into main Jul 15, 2026
2 checks passed
imsujan276 pushed a commit that referenced this pull request Jul 17, 2026
Re-applies PR #143 (reverted in #146) together with the fixes for the
issues found in review. Squashed re-land on top of the revert so the
migration is fully restored (a plain re-merge of the branch would not
re-introduce the reverted commits).

Fixes since the revert:
- CartAddressInput is a Storefront "one of" input; toJson emitted both
  fields, so adding a cart delivery address failed with "'CartAddressInput'
  requires exactly one argument, but 2 were provided". The null field is
  now omitted. (This is the delivery error from the revert note.)
- Example collection tab hard-coded a price {min:500,max:600} filter that
  hid all products on most stores (the "collections show no products" from
  the revert note). Removed it.
- Example Blog/Pages tabs spun forever on a failed fetch (catch cleared the
  flag without setState); they now clear the spinner and show the error or
  an empty state.
- Example builds on the Java 25 / AGP 9 toolchain (Gradle 9.6.1, AGP 9.3.0,
  built-in Kotlin; flutter_inappwebview 6.2.0-beta.3 for AGP 9).
- Dependency floors bumped to current versions (intl kept >=0.20.2 for
  flutter_localizations compatibility).

Verified every migrated document and both delivery paths against a live
2026-07 storefront, ran the example app end-to-end, and confirmed model
parsing on real responses. flutter analyze clean; tests pass.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant