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

Provide a message builder option #357

Closed
1 task
BioPhoton opened this issue Aug 4, 2024 · 3 comments · Fixed by #363
Closed
1 task

Provide a message builder option #357

BioPhoton opened this issue Aug 4, 2024 · 3 comments · Fixed by #363
Assignees
Labels
enhancement New feature or request

Comments

@BioPhoton
Copy link

Thanks for the great lib front off! Really well documented and structured.

Motivation

As a developer I want to have full control of the created error message.
Examples of what I want to control is:

  • all data displayed in the message (e.g. apply different pathoining)
  • text of the displayed message
  • font style of the message (for terminal usage)

Acceptance criteria

  • implement a messageBuilder(issue: ZodIssue): string option so I can have full control of the error.
@jmike
Copy link
Contributor

jmike commented Aug 7, 2024

Yes, that is a good suggestion that would also address #311

I cannot commit on a delivery time, but I would like to work on it.

@jmike jmike added the enhancement New feature or request label Aug 7, 2024
@jmike jmike self-assigned this Aug 18, 2024
@jmike
Copy link
Contributor

jmike commented Aug 18, 2024

@BioPhoton would you be so kind as to review the PR that implements MessageBuilder here (#363)?

If you don't are unable to go through the code, then at least read the docs and let us know if this is what you also had in mind.

@jmike
Copy link
Contributor

jmike commented Sep 13, 2024

I expect the new MessageBuilder to land on npm on the week of Sep 16 (early). Stay tuned.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants