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

Privacy.md #8

Merged
merged 2 commits into from Aug 27, 2018
Merged

Privacy.md #8

merged 2 commits into from Aug 27, 2018

Conversation

HarryHeights
Copy link
Contributor

This is the privacy policy I proposed yesterday. Something's might be off but I'm not entirely sure you'll find any mistakes. Please review and see if there are any modifications to make to the document. Thanks.

This is the privacy policy I proposed yesterday. Something's might be off but I'm not entirely sure you'll find any mistakes. Please review and see if there are any modifications to make to the document. Thanks.
Copy link
Owner

@sanderbaas sanderbaas left a comment

Choose a reason for hiding this comment

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

Looks great, got some minor points.

Privacy.md Outdated

### Usage Data

This app does not record usage information, however, usage statistics are captured by OpenWeatherMap with respect to their [policy](https://openweather.co.uk/privacy-policy).
Copy link
Owner

Choose a reason for hiding this comment

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

Can you maybe change this to the .org url? https://openweathermap.org/privacy-policy

Privacy.md Outdated

## Security Of Data

There are no security risks that may likely occur while using Weather Widget software. However, with the inclusion of OpenWeatherMap API, it is necessary to read how they(OpenWeatherMap) secure your data that may be collected while you utilize Weather Widget. The developers of Phone Saver assume no responsibility for data integrity issues caused by using this application.
Copy link
Owner

Choose a reason for hiding this comment

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

Can you change Phone Saver to Weather Widget?
And maybe add a space after they and before the '(OpenWeatherMap)'?

Privacy.md Outdated

## Changes To This Privacy Policy

This privacy policy statement is not final and is subject to changes at later times. This is done to accommodate changes in the software's architecture due to further developments and that these changes may not align with several statements and policies stated in the document. With that stated, you are advised to check this policy statement occasionally for any changes that may have occurred. Changes to this policy become effective as soon as they are made. Also occasionally check policies of third-party software that is linked with Weather widget.
Copy link
Owner

Choose a reason for hiding this comment

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

Can you uppercase the 'w' in the last word, to keep in line with the other mentions?

Privacy.md Outdated

## Disclaimer

Weather widget is developed in good faith to help end users keep tabs of weather variations at different intervals. The software has an external dependency(OpenWeatherMap API) that might collect some information when you utilize Weather widget. This is not directly controlled by the developer of Weather widget neither does the developer has oversight of the operations of OpenWeatherMap hence, the developer assumes no responsibility for any losses and/or damages of data that are in connection with the use of this software.
Copy link
Owner

Choose a reason for hiding this comment

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

Could you add a space after dependency?

@HarryHeights
Copy link
Contributor Author

All right. I'll make all necessary changes right away.

@HarryHeights
Copy link
Contributor Author

Changes have been made.

@sanderbaas sanderbaas merged commit 487f5c5 into sanderbaas:master Aug 27, 2018
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