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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Saudi Names and Surnames #3

Closed
ozdemirburak opened this issue Oct 14, 2023 · 0 comments 路 Fixed by #34
Closed

Add Saudi Names and Surnames #3

ozdemirburak opened this issue Oct 14, 2023 · 0 comments 路 Fixed by #34
Labels

Comments

@ozdemirburak
Copy link
Owner

ozdemirburak commented Oct 14, 2023

The repository is missing Saudi names and surnames. Follow the guide below to add them.

馃殌 How to Contribute:

  1. Fork the repository.
  2. Add names and surnames for your country following the format provided below.
  3. Submit a Pull Request with your changes.
  4. Please include a reliable source for the names and surnames you're adding, preferably a public database or a reputable website.

馃搫 Format:

For names (src/names/sa.ts):

const saudiNames = {
    0: [ // Male names - Add the URL of the names source here
        'Name1', 'Name2', 'Name3', ...
    ],
    1: [ // Female names - Add the URL of the names source here
        'Name1', 'Name2', 'Name3', ...
    ]
}

export default saudiNames;

For surnames (src/surnames/sa.ts):

const saudiSurnames = [ // Add the URL of the surnames source here
    'Surname1', 'Surname2', 'Surname3', ...
];

export default saudiSurnames;

馃搶 Important Notes:

  • Ensure that the names and surnames you add are common and not specific to a small group.
  • 50 names for both males and females, and 50 surnames.
  • Organize the names with 10 entries per row.
  • Avoid adding names that might be offensive or inappropriate.
  • Ensure you're not violating any copyright or data privacy rules.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant