gumroad icon indicating copy to clipboard operation
gumroad copied to clipboard

Feat: customizable receipts

Open andev18 opened this issue 1 month ago • 1 comments

Resolves https://github.com/antiwork/gumroad/issues/1777

Description of changes

  • Adds Receipt tab and Receipt Preview feature for the user to customize the appearance of the receipt
  • Apply customization to the receipt accordingly from that saved data

The original PR with detailed self-review comments is https://github.com/antiwork/gumroad/pull/1834, I'm opening this new one to address the remaining comments from the reviewer in https://github.com/antiwork/gumroad/pull/1834#pullrequestreview-3525452724

AI Disclosure and confirmation of live-stream PR reviews watching

Cursor (Claude Sonnet 4.5) was used to assist in understanding the issue and implementing the fix. All AI-generated code was reviewed by me.

I've watched the PR review live stream videos.

Result

https://github.com/user-attachments/assets/ac371e91-a347-4f83-b84e-9a6d44aa12d6

UX for light/dark/mobile/desktop

Light Screenshot 2025-11-11 at 12 07 43 AM

Dark Screenshot 2025-11-11 at 12 08 11 AM

Desktop Screenshot 2025-11-11 at 12 07 43 AM

Mobile Screenshot 2025-11-11 at 12 08 49 AM

Tests

Screenshot 2025-11-11 at 12 14 36 AM Screenshot 2025-11-11 at 12 16 04 AM Screenshot 2025-10-22 at 3 02 56 PM

andev18 avatar Dec 01 '25 17:12 andev18

All review comments (from https://github.com/antiwork/gumroad/pull/1834) fixed, except for this one which needs some clarification.

I'll open up the PR as soon as that is sorted.

andev18 avatar Dec 01 '25 18:12 andev18

@EmCousin Could you help clarify this soon? https://github.com/antiwork/gumroad/pull/1834/files#r2578234124

That's the last comment remaining from the previous review 🙏

andev18 avatar Dec 09 '25 15:12 andev18

Opening this up in case @EmCousin can help to clarify https://github.com/antiwork/gumroad/pull/1834/files#r2578234124 during the PR review live-stream!

andev18 avatar Dec 09 '25 16:12 andev18

Can you address these comments? I'll run CI as soon as it's done. Thanks!

@EmCousin Your comments have been addressed, thank you for the kind feedbacks!

andev18 avatar Dec 10 '25 18:12 andev18

@EmCousin I've fixed the nits.

There's only 1 test failure during the CI run which is an unrelated flaky test. Screenshot 2025-12-15 at 9 14 13 PM

It passes just fine when I try running locally Screenshot 2025-12-15 at 9 14 46 PM

I think we're good to merge now.

andev18 avatar Dec 15 '25 14:12 andev18