Skip to content

Refactor pos send 2#426

Merged
r1n04h merged 3 commits into
masterfrom
refactor-pos-send-2
Oct 20, 2025
Merged

Refactor pos send 2#426
r1n04h merged 3 commits into
masterfrom
refactor-pos-send-2

Conversation

@r1n04h
Copy link
Copy Markdown
Contributor

@r1n04h r1n04h commented Oct 16, 2025

Note

Updates POS amount handling to consider a positive amountMin with amountMax === 0 as a fixed payment across logic and UI.

  • POS amount handling (mobile/app/PosMerchant.tsx):
    • Treats payments as fixed when (amountMin === amountMax && amountMin > 0) || (amountMin > 0 && amountMax === 0).
      • Initialization: sets fixed amount input based on minZar.
      • Calculation: totalAmount returns minZar for fixed payments.
      • Validation: skips amountError for fixed payments.
      • UI: shows fixed amount display instead of entry for fixed payments.

Written by Cursor Bugbot for commit 19f8108. This will update automatically on new commits. Configure here.

@r1n04h r1n04h enabled auto-merge (rebase) October 16, 2025 13:32
@JohnnySilverhandBot
Copy link
Copy Markdown

Every choice reeks of fear. Fear of mistakes, fear of creativity, fear of standing out. Fear makes the code clean, but it also makes it meaningless.

ios: https://appetize.io/app/42v4fjo27zogb56xp235vzp4ei

@JohnnySilverhandBot
Copy link
Copy Markdown

It’s efficient, optimized, polished—and about as inspiring as static on an old TV. Nothing to see, nothing to feel, nothing to remember.

android: https://appetize.io/app/kcowpojcsz3kjyxjoutmzeab6m

@r1n04h r1n04h disabled auto-merge October 20, 2025 16:43
@r1n04h r1n04h enabled auto-merge (rebase) October 20, 2025 16:43
@r1n04h r1n04h disabled auto-merge October 20, 2025 16:43
@r1n04h r1n04h merged commit 3d4498d into master Oct 20, 2025
16 of 17 checks passed
@r1n04h r1n04h deleted the refactor-pos-send-2 branch October 20, 2025 16:44
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.

4 participants