search

BlackBoxVision / react-admin-extensions

Packages for improving your development workflow with react admin
MIT License
32 stars 18 forks source link

Updated spanish translation #22

Open ddanielroche opened 3 months ago

ddanielroche commented 3 months ago

Hi Team

These are the changes that need to be made after reviewing the entire Spanish translation file.

I take the initiative and here is my contribution @manutuero

This is my personal user on github, @dany-loveevents is the company user.

I would like to leave the issue of ra-core and extension versions to you.

Thanks!

manutuero commented 2 months ago

Hi @ddanielroche,

Thanks for your initiative and contribution to the Spanish translation. I really appreciate your effort.

I’m currently tied up with a major project delivery, so I won’t have time to work on this for the next few weeks.

Since you’ve already started, could you continue with the modifications? I’ll review everything once you’re done.

Thanks for understanding and your help.

Best,

ddanielroche commented 2 months ago

Hi @manutuero

No problem.

I was thinking that I should first merge this change and generate a version of the extension for read admin 4, it was in version 4 that I found the missing variable. Then I would generate a new version compatible with read admin 5.

That way it would be compatible with both versions. Read Admin 5 is very new and there will be many projects developed in 4 that would benefit from receiving this change.

What do you think?

Thanks!

manutuero commented 2 months ago

Hi @ddanielroche

Perfect. I think that it's a good idea.