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

feat: Added a ISO639-1 decorator #2047

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Conversation

Gustrb
Copy link

@Gustrb Gustrb commented Apr 15, 2023

This decorator is used to check if a string matches the ISO 639-1, that describes how country languages prefixes should look like.
Fortunatelly, it just delegates the validation to an already existent function in the validator library

This PR implements issue #2045

This decorator is used to check if a string matches
the ISO 639-1, that describes how country languages prefixes
should look like.
Fortunatelly, it just delegates the validation to an already existent
function in the validator library
@ytetsuro
Copy link
Contributor

ytetsuro commented Apr 22, 2023

It would be great if you could update the README.md as well!
Please add a test.

Copy link
Contributor

@rubiin rubiin left a comment

Choose a reason for hiding this comment

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

Add tests as well as update the readme

}

/**
* Checks if the string is an ISIN (stock/security identifier).

Choose a reason for hiding this comment

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

Copy/paste error from other decorator.

@te-online
Copy link

@Gustrb Any update on your PR? This looks like a nice addition 😊

@Gustrb
Copy link
Author

Gustrb commented Apr 30, 2024

@te-online, I kind of forgot about it.
I added here since I needed this function inside the library, but I hacked my way into it a few months ago

Will work on adding some tests and fixing the comments pointed out :P

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants