Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Square POS changes #1265

Merged
merged 11 commits into from
Jan 26, 2023
Merged

Square POS changes #1265

merged 11 commits into from
Jan 26, 2023

Conversation

kaloudis
Copy link
Contributor

@kaloudis kaloudis commented Jan 22, 2023

In our initial implementation, we were using Square's Order API which is meant mostly for web payments. When using Square Restaurants, checks are left open and not synced with the API until they are marked paid.

To trigger payment with ZEUS, servers must mark the payment as made with Other Payment Type and apply the note including the text 'Zeus', 'BTC', or 'Bitcoin' (case insensitive).

Tips now are only recorded in ZEUS as we cannot modify an order in the Square API once marked Completed -- which unfortunately is the only state we can send a check to. See

Other changes:

  • Users can now set their tip amounts in BTC or sats, if they choose not to denominate in fiat
  • Open and Paid orders are now split up
  • Tip amounts are now displayed on the paid orders list
  • Paid orders return recorded tip amounts and associated TX IDs or payment requests (on-chain v LN)
  • Merchant now has the ability to disable tips (helpful if they are applying tip on the Square terminal / adding auto-gratuity)
  • Merchants can now set their merchant name to appear in LN memos

@kaloudis kaloudis marked this pull request as ready for review January 26, 2023 02:39
@kaloudis kaloudis merged commit 4bf79f1 into ZeusLN:master Jan 26, 2023
@kaloudis kaloudis deleted the pos-changes branch January 26, 2023 02:42
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