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

[l10n] Improve Spanish (es-ES) locale #15369

Merged
merged 3 commits into from
Nov 15, 2024

Conversation

CarlosLopezLg
Copy link
Contributor

@CarlosLopezLg CarlosLopezLg commented Nov 10, 2024

@mui-bot
Copy link

mui-bot commented Nov 10, 2024

Localization writing tips ✍️

Seems you are updating localization 🌍 files.

Thank you for contributing to the localization! 🎉 To make your PR perfect, here is a list of elements to check: ✔️

  • Verify if the PR title respects the release format. Here are two examples (depending if you update or add a locale file)

    [l10n] Improve Swedish (sv-SE) locale
    [l10n] Add Danish (da-DK) locale

  • Update the documentation of supported locales by running pnpm l10n
  • Clean files with pnpm prettier.

Deploy preview: https://deploy-preview-15369--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against 7cd1238

@zannager zannager added the l10n localization label Nov 11, 2024
@KenanYusuf
Copy link
Member

Hey @CarlosLopezLg, thanks for the PR. Please can you update it to compare against master instead of v7.x?

Also, there are some additional steps to follow in this comment: #15369 (comment)

@alexfauquette alexfauquette changed the title Update esES.ts [l10n] Improve Spanish (es-ES) locale Nov 14, 2024
@alexfauquette alexfauquette changed the base branch from v7.x to master November 14, 2024 09:20
@alexfauquette alexfauquette changed the base branch from master to v7.x November 14, 2024 09:20
CarlosLopezLg and others added 2 commits November 14, 2024 10:23
Signed-off-by: CarlosLopezLg <101943016+CarlosLopezLg@users.noreply.github.com>
@alexfauquette alexfauquette changed the base branch from v7.x to master November 14, 2024 09:23
@alexfauquette alexfauquette added the needs cherry-pick The PR should be cherry-picked to master after merge label Nov 15, 2024
@oliviertassinari oliviertassinari merged commit b797d1f into mui:master Nov 15, 2024
20 checks passed
oliviertassinari pushed a commit that referenced this pull request Nov 15, 2024
Signed-off-by: CarlosLopezLg <101943016+CarlosLopezLg@users.noreply.github.com>
Co-authored-by: alex <alex.fauquette@gmail.com>
@oliviertassinari
Copy link
Member

@CarlosLopezLg It's a great first pull request on this project 👌🏻. Thank you for working on it!

@LukasTy LukasTy added the component: data grid This is the name of the generic UI component, not the React module! label Nov 18, 2024
LukasTy pushed a commit to LukasTy/mui-x that referenced this pull request Dec 19, 2024
Signed-off-by: CarlosLopezLg <101943016+CarlosLopezLg@users.noreply.github.com>
Co-authored-by: alex <alex.fauquette@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: data grid This is the name of the generic UI component, not the React module! l10n localization needs cherry-pick The PR should be cherry-picked to master after merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants