feat: [CIT-5223] bump @kiva/kv-shop to ^3.8.32 - #7267
Merged
Merged
Conversation
3.8.32 adds checkout_in_progress and shop.checkoutInProgress handling to parseShopError. Without it, a lender applying or removing Kiva credit during their own in-flight checkout sees "An unknown error occurred." from express checkout, because those paths run through kv-shop's callShopMutation rather than src/util/basketUtils.js. Verified the published build carries the fix before bumping. CIT-5223 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
dyersituations
approved these changes
Sep 16, 2026
JoeTravisKiva
changed the title
JoeTravisKiva
marked this pull request as ready for review
September 17, 2026 16:42
kiva-robot
added a commit
that referenced
this pull request
Sep 22, 2026
## [3.96.0-rc.1](v3.95.0...v3.96.0-rc.1) (2026-09-22) ### 🎉 New Features * [MP-3208] - done show goal days left if there 90 days or less t… ([#7265](#7265)) ([f22f4b9](f22f4b9)) * add fundraisers entry point to MyKiva ([#7271](#7271)) ([7b25967](7b25967)) * additional copy edits ([9eeac5f](9eeac5f)) * additional merge fix ([191afb1](191afb1)) * changed copy ([244ce6d](244ce6d)) * corrected revisions ([294d6b8](294d6b8)) * edited unit tests ([2f6e28b](2f6e28b)) * gate the MyKiva fundraisers row behind an experiment ([#7274](#7274)) ([a374c8d](a374c8d)) * increased padding ([4a57d64](4a57d64)) * mp-3158 track the non-express checkout path from lend after goal set ([#7260](#7260)) ([81db98e](81db98e)) * mp-3209 keep completed goal at front until its goal year ends ([#7269](#7269)) ([57d81e3](57d81e3)) * padding only on express checkout context ([7d0f56a](7d0f56a)) * removed img file + label context; prop binding fix ([f6d60f5](f6d60f5)) * revised title format and spec tests context ([d641017](d641017)) * simplified expression ([75119c4](75119c4)) ### 🐛 Bugfixes * **build:** finish the ssr-manifest-css plugin to match quasar's fix ([75638bd](75638bd)) * **build:** link the stylesheets a chunk reaches by static import ([af935d9](af935d9)) * ensure goal summary query is not fired if the user does not meet the conditions to see goal in recap ([575656d](575656d)) * mp-3209 make the recapDate override reach the server so hydration matches ([#7278](#7278)) ([a0e000e](a0e000e)) * reduce duplicate giving fund queries on portfolio ([#7272](#7272)) ([34e2e93](34e2e93)) * rephrase badly worded comments ([ad1b7df](ad1b7df)) * show real copy when the basket is locked by an in-flight checkout ([#7266](#7266)) ([b4944cd](b4944cd)) ### 🪚 Refactors * drop the isEligible argument from shouldAutoOpenRecap ([3c1d786](3c1d786)) * look up the spotlight category by slug ([#7268](#7268)) ([7045ee1](7045ee1)) * reuse shouldAutoOpenRecap instead of splitting it ([8c7cc15](8c7cc15)) ### 🔍 Tests * missing coverage test ([b44f681](b44f681)) ### 🧹 Chores * **deps:** bump @kiva/kv-shop to ^3.8.32 ([#7267](#7267)) ([1194398](1194398)) * **deps:** upgrade kiva packages to latest versions AD-501 AD-506 ([1cda5df](1cda5df)) * drop stale dependabot ignores and auto-assign workflow ([#7273](#7273)) ([a6fde5e](a6fde5e))
Collaborator
|
🎉 This PR is included in version 3.96.0-rc.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
kiva-robot
added a commit
that referenced
this pull request
Sep 23, 2026
## [3.96.0](v3.95.0...v3.96.0) (2026-09-23) ### 🎉 New Features * [MP-3208] - done show goal days left if there 90 days or less t… ([#7265](#7265)) ([f22f4b9](f22f4b9)) * add fundraisers entry point to MyKiva ([#7271](#7271)) ([7b25967](7b25967)) * additional copy edits ([9eeac5f](9eeac5f)) * additional merge fix ([191afb1](191afb1)) * changed copy ([244ce6d](244ce6d)) * corrected revisions ([294d6b8](294d6b8)) * edited unit tests ([2f6e28b](2f6e28b)) * gate the MyKiva fundraisers row behind an experiment ([#7274](#7274)) ([a374c8d](a374c8d)) * increased padding ([4a57d64](4a57d64)) * mp-3158 track the non-express checkout path from lend after goal set ([#7260](#7260)) ([81db98e](81db98e)) * mp-3209 keep completed goal at front until its goal year ends ([#7269](#7269)) ([57d81e3](57d81e3)) * padding only on express checkout context ([7d0f56a](7d0f56a)) * removed img file + label context; prop binding fix ([f6d60f5](f6d60f5)) * revised title format and spec tests context ([d641017](d641017)) * simplified expression ([75119c4](75119c4)) ### 🐛 Bugfixes * **build:** finish the ssr-manifest-css plugin to match quasar's fix ([75638bd](75638bd)) * **build:** link the stylesheets a chunk reaches by static import ([af935d9](af935d9)) * ensure goal summary query is not fired if the user does not meet the conditions to see goal in recap ([575656d](575656d)) * mp-3209 make the recapDate override reach the server so hydration matches ([#7278](#7278)) ([a0e000e](a0e000e)) * reduce duplicate giving fund queries on portfolio ([#7272](#7272)) ([34e2e93](34e2e93)) * rephrase badly worded comments ([ad1b7df](ad1b7df)) * show real copy when the basket is locked by an in-flight checkout ([#7266](#7266)) ([b4944cd](b4944cd)) ### 🪚 Refactors * drop the isEligible argument from shouldAutoOpenRecap ([3c1d786](3c1d786)) * look up the spotlight category by slug ([#7268](#7268)) ([7045ee1](7045ee1)) * reuse shouldAutoOpenRecap instead of splitting it ([8c7cc15](8c7cc15)) ### 🔍 Tests * missing coverage test ([b44f681](b44f681)) ### 🧹 Chores * **deps:** bump @kiva/kv-shop to ^3.8.32 ([#7267](#7267)) ([1194398](1194398)) * **deps:** upgrade kiva packages to latest versions AD-501 AD-506 ([1cda5df](1cda5df)) * drop stale dependabot ignores and auto-assign workflow ([#7273](#7273)) ([a6fde5e](a6fde5e)) * **release:** 3.96.0-rc.1 [skip ci] ([1c45283](1c45283)), closes [#7265](#7265) [#7271](#7271) [#7274](#7274) [#7260](#7260) [#7269](#7269) [#7278](#7278) [#7272](#7272) [#7266](#7266) [#7268](#7268) [#7267](#7267) [#7273](#7273)
Collaborator
|
🎉 This PR is included in version 3.96.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CIT-5223 — the ui half. The kv-shop change is kiva/kv-ui-elements#905, merged and released as 3.8.32.
Why
Express checkout's credit controls (
ExpressCheckoutTotals.vue→applyKivaCredit/removeKivaCredit) run through kv-shop'scallShopMutation, notsrc/util/basketUtils.js— so CIT-5181 (#7266) does not reach them.Before 3.8.32,
parseShopErrorhad no mapping forcheckout_in_progress/shop.checkoutInProgress, so both hit its catch-all and returned "An unknown error occurred." with the original code rewritten toshop.unknown. That rewrite is why it could not be fixed from ui — there was nothing left to classify on.What
One line in
package.json, plus the lockfile. The lockfile diff touches only@kiva/kv-shop— no other package moved.Verification
npm installNotes
node_moduleswas stale before this (3.8.9 installed against a lockfile pinning 3.8.28). The lockfile itself was fine; only the local install had drifted, andnpm installresolved it. Worth anpm cilocally if you had the same drift.🤖 Generated with Claude Code