Skip to content

feat: 2 new user statuses. percentage schema and type#166

Merged
sergiojoker11 merged 2 commits into
mainfrom
feat/stuff-for-subscriptions
Jun 27, 2025
Merged

feat: 2 new user statuses. percentage schema and type#166
sergiojoker11 merged 2 commits into
mainfrom
feat/stuff-for-subscriptions

Conversation

@sergiojoker11
Copy link
Copy Markdown
Member

No description provided.

Comment thread src/schemas/common.ts Outdated
.string()
.max(128)
.transform((data) => data as TimeZone);
export const percentageSchema = z
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hmmmmm I'd argue that 150% of something is a valid percentage...

Maybe rename it to strictPercentageSchema?

Or ignore me xD

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I have removed the upper limit, it made no much sense

dsiguero
dsiguero previously approved these changes Jun 26, 2025
@sergiojoker11 sergiojoker11 force-pushed the feat/stuff-for-subscriptions branch from 5fc8e12 to fc529e8 Compare June 27, 2025 11:59
@sergiojoker11 sergiojoker11 merged commit 2fa5ee6 into main Jun 27, 2025
2 checks passed
@sergiojoker11 sergiojoker11 deleted the feat/stuff-for-subscriptions branch June 27, 2025 12:01
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.

2 participants