-
-
Notifications
You must be signed in to change notification settings - Fork 491
Add privacy policy menu item #883
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest switching places between the Help > Privacy Policy and Help > Check for Arduino IDE updates menu items.
It seems more logical to me to have all the website visit menu items grouped together, rather than having the very different Check for Arduino IDE updates menu item interspersed.
Before:
- Help
- Getting Started
- Environment
- Troubleshooting
- Reference
- Find in Reference
- Frequently Asked Questions
- Visit Arduino.cc
- Check for Arduino IDE Updates
- Privacy Policy
- About Arduino IDE
- Toggle Developer Tools
After:
- Help
- Getting Started
- Environment
- Troubleshooting
- Reference
- Find in Reference
- Frequently Asked Questions
- Visit Arduino.cc
- Privacy Policy
- Check for Arduino IDE Updates
- About Arduino IDE
- Toggle Developer Tools
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should the URL be an internationalized string (
nls.localize('https://www.arduino.cc/en/privacy-policy')) so that it can be pointed at the appropriate page (e.g., https://www.arduino.cc/it/privacy-policy)?I tried changing my browser locale to
itin order to see whether arduino.cc automatically redirects https://www.arduino.cc/en/privacy-policy to https://www.arduino.cc/it/privacy-policy, and it didn't, but I don't know much about the subject.
@per1234 I thought about that too, but seeing the page was in English-only I didn't localize the URL. @ubidefeo, do you think we should ask someone to translate that page?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@AlbyIanna there is an Italian language version of the page:
https://www.arduino.cc/it/privacy-policy
I don't think there are any others at the moment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I swear it didn't work the first time I tried that URL 😅 It just redirected me to the English version of the page. Weird.
Anyway, should we localize it? I don't know, I think it'd be kinda strange if we'd localize it only for Italian folks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't personally find it strange, but I also don't have strong feelings about the subject either way. I added the question about the internationalization only as a comment, and not as part of my review for that reason.
I'll go ahead and submit an approving review now since the review suggestion I made has been resolved (thanks!) and we are on the same page regarding the existence of an Italian version of the page.
I'll leave the decision on whether to internationalize that URL string to you or any other interested parties.
d3e5dce to
352ec6b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Motivation
We need to add a link to our Privacy Policy
Reviewer checklist