Change error massage param to function

[vicimpa]

For multi language. Example:

const LoginSchema = object({
  email: string([
    minLength(1, () => 'Please enter your email.'),
    email(() => 'The email address is badly formatted.'),
  ]),
  password: string([
    minLength(1, () => 'Please enter your password.'),
    minLength(8, () => 'Your password must have 8 characters or more.'),
  ]),
});

[netlify]

✅ Deploy Preview for valibot ready!

Name	Link
🔨 Latest commit	0bcdca1
🔍 Latest deploy log	https://app.netlify.com/sites/valibot/deploys/64f22fbc5d6ff1000857d68b
😎 Deploy Preview	https://deploy-preview-136--valibot.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[fabian-hiller]

Thank you very much! Do you have an opinion on this issue? #36

[vicimpa]

Yes, sure. See.

const LoginSchema = object({
  email: string([
    minLength(1, () => i18n["min_length"]),
    email(() => i18n["email_fail"]),
  ])
});

[ragokan]

Just wanted to comment, that looks pretty good for localization.

[fabian-hiller]

I haven't made a decision about i18n yet. If you have a clear opinion, feel free to tell more about why this solution is better than the other ones in #36.

[vicimpa]

My solution is more universal and is not tied to the i18n library

[thundermiracle]

Is there anyone who agrees that passing messages or functions to every validation function is inconvenient as I do?

[fabian-hiller]

@thundermiracle are you interested in a global solution to pass individual error messages?

[thundermiracle]

@thundermiracle are you interested in a global solution to pass individual error messages?

Yes, of course. What's the solution?

[fabian-hiller]

There are several ideas in issue #36. Which one do you prefer?

[thundermiracle]

There are several ideas in issue #36. Which one do you prefer?

I'm not sure. But

1. Passing functions or messages to every function is not preferable
2. Adding global state as gmaxlev suggested is not that good in functional programming

What about record map which support both string and function which others suggested?

Maybe we should discuss it in #36 ?

[fabian-hiller]

Yes, please share your ideas and feedback there. I plan to work on an implementation next week.

[fabian-hiller]

Thanks again for creating this PR. Since I would do some basic things differently and it was faster for me to implement them directly, I made the changes independently. However, I have added you as a co-author and linked the PR in the changelog.