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

Spanish language pack #82

Merged
merged 4 commits into from
Dec 14, 2021
Merged

Spanish language pack #82

merged 4 commits into from
Dec 14, 2021

Conversation

JonL1
Copy link
Contributor

@JonL1 JonL1 commented Dec 9, 2021

No description provided.

@MrWook MrWook added the enhancement New feature or request label Dec 10, 2021
@MrWook
Copy link
Collaborator

MrWook commented Dec 10, 2021

Hey, thanks for your contribution with the new language package.

I appreciate your efforts with the list but could you add the sources to the list generator like descript in the documentation
https://zxcvbn-ts.github.io/zxcvbn/guide/languages/#add-a-new-language-package.
This way everybody nows where all this data is coming from and maybe can be updated in the future if needed.

packages/languages/es/src/index.ts Outdated Show resolved Hide resolved
packages/languages/es/package.json Outdated Show resolved Hide resolved
packages/languages/es/package.json Outdated Show resolved Hide resolved
packages/languages/es/package.json Outdated Show resolved Hide resolved
packages/languages/es/README.md Outdated Show resolved Hide resolved
@JonL1
Copy link
Contributor Author

JonL1 commented Dec 10, 2021

Hey, thanks for your contribution with the new language package.

I appreciate your efforts with the list but could you add the sources to the list generator like descript in the documentation https://zxcvbn-ts.github.io/zxcvbn/guide/languages/#add-a-new-language-package. This way everybody nows where all this data is coming from and maybe can be updated in the future if needed.

It was all done manually as our government doesn't provide with an easy format to extract from. Is it OK if I create a repo with the raw files and add the information in the repo from where and how they were extracted?

I will correct the rest of naming errors.

@JonL1
Copy link
Contributor Author

JonL1 commented Dec 10, 2021

I found something for the generators. I'll work on that.

@MrWook
Copy link
Collaborator

MrWook commented Dec 10, 2021

I would be fine if you create a github repository and explain where you got them from.
But if you have something else even better :)

@JonL1
Copy link
Contributor Author

JonL1 commented Dec 11, 2021

The generators work correctly but we should add some pre-processing function to filter compound names.

Spanish (ab)uses compound first names. So instead of being named "Antonio", "Jose", "Pedro" your parents choose to call you "Antonio Manuel", "Jose Miguel" or "Pedro Antonio" to honour(keep happy) other family members.

And as this is quite popular the list of first names given by the government is plagued by such. For the purpose of the matcher this is useless as any of the compound names will already match the single name from the list.

I'm sending a new PR for the Spanish language pack including compound names and I open a new issue on how to deal with compound names.

@JonL1
Copy link
Contributor Author

JonL1 commented Dec 11, 2021

As mentioned here wikipedia data is not included in my commit.

Added an empty wikipedia.json. I hope you can process it on your side as I'm currently on mobile data. If I download the wikipedia data it will make a huge dent on my limits :)

@MrWook
Copy link
Collaborator

MrWook commented Dec 14, 2021

I'm sorry the wikipedia list needs to return an empty array 🙈

@JonL1
Copy link
Contributor Author

JonL1 commented Dec 14, 2021

Done.

@MrWook MrWook merged commit e6ef77f into zxcvbn-ts:master Dec 14, 2021
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 this pull request may close these issues.

None yet

2 participants