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

Update localization.h #117

Merged
merged 11 commits into from
Mar 31, 2024
Merged

Update localization.h #117

merged 11 commits into from
Mar 31, 2024

Conversation

klamath1977
Copy link
Contributor

Add Italian Language Setting

Add Italian Language Setting
Add Italian Language
Add Italian Language but I prefer portal in english
Add Italian Language Setting
commit bc8928b
Author: klamath <klamath1977@gmail.com>
Date:   Sun Feb 25 20:59:19 2024 +0100

    Update portal.h

    Add Italian Language but I prefer portal in english

commit 96786a3
Author: klamath <klamath1977@gmail.com>
Date:   Sun Feb 25 20:52:22 2024 +0100

    Update localization.h

    Add Italian Language Setting
minor change
Copy link
Owner

@n0xa n0xa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it. Can you limit the scope of the PR to:

  • Addition of Italian to localization.h
  • Italian support to portal.h
  • The italian define checks in portal.h and m5stick-nemo.ino (like 9 and 28 respectively)
  • Wrap the wifi channel list in a language check so that it's 1-11 if LANGUAGE_EN_US is defined, and 1-14 otherwise - I do not want NEMO to break FCC laws in USA.

Also, Please base your PR against develop branch and please ensure you pick up some recently added texts such as color selection which I added to EN_US and PT_BR already.

	modificato:             wifispam.h
@n0xa n0xa changed the base branch from main to develop March 31, 2024 04:53
@n0xa n0xa merged commit 7d8d256 into n0xa:develop Mar 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants