Skip to content

Conversation

@govind-eka
Copy link
Collaborator

No description provided.

@govind-eka govind-eka changed the title Receipt upgrades 1 Receipt upgrades, total in words, date and time + bill created at [FE - 2] Jan 1, 2026
ageInM?: number;
gender?: string;
payment_status: string;
bill_created_at?: string;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

p1: consistency in key names: either _ or camelCase. We can pick any one and go ahead

payment_status: string;
bill_created_at?: string;
receiptAmountinWords?: string;
totalDiscountOnEntireBill: number;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why it's not optional?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

product! CORE FEATURE NOW!!
TOTAL DISCOUNT will always be there. provided it respects the config.
hence pdf object constructor that is pre pagify step/elixir will ALWAYS pass this! total discount

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cool

Copy link
Collaborator

@jainkuniya jainkuniya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test & go

@jainkuniya jainkuniya merged commit 851ad0d into main Jan 6, 2026
0 of 2 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.

3 participants