Skip to content

Fix bounty payout memo text#2790

Open
cybercraftsolutionsllc wants to merge 1 commit into
stakwork:masterfrom
cybercraftsolutionsllc:codex/bounty-payment-memo-993
Open

Fix bounty payout memo text#2790
cybercraftsolutionsllc wants to merge 1 commit into
stakwork:masterfrom
cybercraftsolutionsllc:codex/bounty-payment-memo-993

Conversation

@cybercraftsolutionsllc
Copy link
Copy Markdown

Summary

  • fixes bounty payout memos so keysend notifications include a clean bounty title instead of the current trailing s typo
  • centralizes bounty payment memo formatting with an empty-title fallback
  • refreshes payment test expectations and the generated database mock needed for handler tests to compile

Fixes #993.

Validation

  • go test ./utils -run 'TestBuildBountyPaymentMemo|TestBuildV2KeysendBodyData' -count=1
  • go test ./handlers -run '^$' -count=1
  • git diff --check

go test ./handlers -run TestMakeBountyPayment -count=1 was attempted, but the local test database was not running at localhost:5532, so that integration-style test could not connect.

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.

Investigate including keysend message for bounty payouts

2 participants