Add support for ULID validation

simoneNEMO commented 5 months ago

❓ Type of change

[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[X] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

ULID is a widely used format for identifiers with some advantages over the canonically recognised UUID formats (1-5). ULIDs are natively supported in multiple frameworks, e.g. Laravel.

This pull request introduces support for ULID validation in VineJS by means of simple regex matching.

ULID specs: https://github.com/ulid/spec

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Haven't been able to open a simple issue, only the "bug template" seems to work. If you consider to include this change I can update the docs accordingly.

thetutlage commented 4 months ago

Hey, can we please have some tests for the isUlid helper?

simoneNEMO commented 4 months ago

Hey, can we please have some tests for the isUlid helper?

Added some integration tests, tests for the isUlid helper and rebased

RomainLanz commented 4 months ago

Looks great! Thanks for the PR!

Do you mind creating another PR in the documentation repository?

simoneNEMO commented 4 months ago

Do you mind creating another PR in the documentation repository?

Thanks for merging, will do during the day!

simoneNEMO commented 4 months ago

@RomainLanz Added PR to docs here: https://github.com/vinejs/vinejs.dev/pull/22

vinejs / vine