Skip to content

Remove Shop production warning banners and auth capability dumps - #76

Merged
BitcoinErrorLog merged 9 commits into
release/shop-v0.6.8from
cursor/remove-warnings-3900
Sep 22, 2026
Merged

BitcoinErrorLog merged 9 commits into
release/shop-v0.6.8from
cursor/remove-warnings-3900

Conversation

@BitcoinErrorLog

@BitcoinErrorLog BitcoinErrorLog commented Sep 22, 2026

Copy link
Copy Markdown
Owner

Do not merge. Inventory of disclaimer/warning copy on Shop marketplace surfaces, plus removal of OUR unsolicited banners and auth QR capability-path dumps.

Removed (ours)

Surface Copy removed Blame
How you get paid (Locks/Paykit amber) "Locks and Paykit Server are pre-production. Do not use this prototype…" 2fd94c9dd8 Bitcoin Error Log 2026-08-20
How you get paid subtitle "Every method pays the seller directly — this marketplace never holds funds." Shop payment-settings intro
Checkout production amber "Real money. Payments are final and go directly to the seller." 7a7a3c00a / 014774c89
Order Awaiting payment amber same Real money banner same
Order Awaiting payment helper "Choose how to pay. Every method pays the seller directly." holderUnboundCopy
Sign-in QR capability path dump + "compare it before approving" c213ae9f36 2026-09-08
Reauth dialog verbatim requested capabilities e2975e6dd6 2026-09-05
Session connect / grant QR path dump + empty-permission-list copy c213ae9f36 / 13c13da94
Messaging enable QR Paykit capability path dump Shop messaging enable
Get paid technical details "exact creator capability grant" Paykit setup details

Kept

Surface Copy Why
Catalog / cart / order (staging only) "Staging environment — test rails, no real funds move" environment warning, getDeployEnv() === 'staging'
Sandbox badge "Sandbox · simulated payment · no real funds" sandbox fact
Guarantee checkbox "this marketplace never holds funds" inside accept-terms legal accept-terms, not an amber callout
Checkout muted helper (production, not sandbox) "Paid directly to the seller." one-line fold-in under Place order
Awaiting payment muted helper "The item is held for you once a payment starts." / "…until {deadline}." hold/reservation fact
Processor / PayPal-verified badges verification fact product state
How you get paid badge "Seller-direct payouts" product label

Added helper (Paykit Step 2)

Muted paragraph on Seller Studio Paykit setup Step 2 and the Connect Bitkit dialog (not a warning box):

Your Shop identity must live in Bitkit. Signed up with Pubky Ring? Create a new Shop account by scanning the sign-up QR with Bitkit — Ring import is coming to Bitkit.

Bitkit import of Ring identities is synonymdev/bitkit-android#1109, not in Bitkit master. Same guidance + Bitkit QR workaround posted on marketplace#53.

Not ours — leave

Surface Copy Blame
DialogTerms beta upstream Synonym DialogTerms Flavio 2025-09-01

Grep

Real money / Choose how to pay / pays the seller directly are gone from src/ product surfaces. never holds funds remains only on the guarantee checkbox accept-terms in MarketplaceCart.tsx.

VRT

Rerecorded for this helper: payment-settings-bitkit-dialog-mobile (darwin chromium/firefox/webkit + linux chromium/firefox) and payment-settings-display-preferences-desktop (Step 2 is in that frame). Path argument does not isolate --update; extras were reverted.

Drop unsolicited maturity and real-money amber callouts from checkout,
orders, and seller payment settings. Staging keeps the test-rails notice.
Awaiting payment keeps a single muted hold sentence. Sign-in and grant
QR dialogs no longer dump capability paths.
The QR no longer dumps capability paths. Auction bid step-up now asserts
the plain Shop sentence instead of the old permission-list copy.
…lines

Point the awaiting-payment method-picker scene at production so it
proves the Real money amber is gone and only the muted hold sentence
remains. Commit regenerated darwin baselines for cart, messaging,
payment settings, Locks awaiting-start, and session-connect.
Awaiting payment shows only the muted hold sentence, with no Real money
amber. Darwin and linux method-picker scenes were deleted and recorded
fresh so the 2% marketplace comparator cannot hide the copy change.
Paykit setup Step 2 and the Connect Bitkit dialog now carry a muted helper: Shop identity must live in Bitkit, and Ring import is not available yet.
…aselines

Step 2 and the Connect Bitkit dialog gained a muted Ring-identity helper. Darwin and linux scenes that include that copy are rerecorded; extras were not kept.
…/merge-seq-76-8b74

# Conflicts:
#	src/components/organisms/Marketplace/MarketplaceSessionConnectDialog.tsx
…/merge-seq-76-8b74

# Conflicts:
#	src/components/templates/Marketplace/MarketplaceCart.tsx
#	src/test/vrt/marketplace/__screenshots__/MarketplaceCart.vrt.test.tsx/cart-locks-paykit-desktop-chromium-darwin.png
#	src/test/vrt/marketplace/__screenshots__/MarketplaceCart.vrt.test.tsx/cart-locks-paykit-desktop-firefox-darwin.png
#	src/test/vrt/marketplace/__screenshots__/MarketplaceCart.vrt.test.tsx/cart-locks-paykit-desktop-webkit-darwin.png
@BitcoinErrorLog
BitcoinErrorLog marked this pull request as ready for review September 22, 2026 08:21
Copy the 17 *-actual-*-linux.png files from Marketplace VRT run
35704148854 onto the matching __screenshots__ linux baselines so
vrt-marketplace matches the current session-connect, cart, messaging,
payment-settings, and payment-status scenes after the #73 merge.
@BitcoinErrorLog
BitcoinErrorLog merged commit 5417062 into release/shop-v0.6.8 Sep 22, 2026
14 checks passed
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