search

nielsfaber / alarmo-card

Home Assistant card for controlling the Alarmo component
108 stars 20 forks source link

Add pt-br language #83

Closed rodolfomenardi closed 2 months ago

rodolfomenardi commented 4 months ago

I added the Brazilian Portuguese language file., but I'm not a frontend dev and I don't know much about javascript/node/npm. I couldn't build the application to test, I always get this error: image

After changing the .eslintrc file to .eslintrc.js, it passed this step, but it reports a lint error in other files that I didn't change: image

I tried ignoring the lint and just running the build to test, but I get the following error: image

Could you help me before incorporating the translation?

I don't know if it could be the versions I'm using or something. I'm using:

nodejs: 18.19.1

github-actions[bot] commented 3 months ago

This pull request is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days

rodolfomenardi commented 3 months ago

Not close

github-actions[bot] commented 2 months ago

This pull request is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days

nielsfaber commented 2 months ago

Sorry for the delay. I don't really know how to help you build the project. For me NPM is also a bit of magic and I have been struggling setting this up. The translation looks fine to me, so we can merge and release it. If changes need to be made, you can submit a new PR for that.