Per the most recent activity on the old PR #1528 I have reconfigured this request to make the required changes.
Given that Mullvad now has published Apps on both main app stores for Android and iOS, I have included links and corrected the incorrect documentation that mobile clients were still in the works.
Resolves: #1528 which was erroneously closed. (By me, oops).
Check List
I understand that by not opening an issue about a software/service/similar addition/removal, this pull request will be closed without merging.
Code repository of the project (if applicable):
N/A
<!-- PLEASE READ OUR CODE OF CONDUCT (https://wiki.privacytools.io/view/PrivacyTools:Code_of_Conduct) AND CONTRIBUTING GUIDELINES (https://github.com/privacytoolsIO/privacytools.io/blob/master/.github/CONTRIBUTING.md) BEFORE SUBMITTING -->
## Description
Per the [most recent activity](https://github.com/privacytoolsIO/privacytools.io/pull/1528#issuecomment-612105347) on the old PR #1528 I have reconfigured this request to make the required changes.
Given that Mullvad now has published Apps on both main app stores for Android and iOS, I have included links and corrected the incorrect documentation that mobile clients were still in the works.
Resolves: #1528 which was erroneously closed. (By me, oops).
#### Check List <!-- Please add an x in each box below, like so: [x] -->
- [x] I understand that by not opening an issue about a software/service/similar addition/removal, this pull request will be closed without merging.
- [x] I have read and understand [the contributing guidelines](https://github.com/privacytoolsIO/privacytools.io/blob/master/.github/CONTRIBUTING.md).
- [x] The project is [Free Libre](https://en.wikipedia.org/wiki/Free_software) and/or [Open Source](https://en.wikipedia.org/wiki/Open-source_software) Software
* Netlify preview for the mainly edited page: <!-- link or Non Applicable? Edit this in afterwards -->
[Netlify Preview here](https://eager-colden-4cbe67.netlify.com/providers/vpn/)
* Code repository of the project (if applicable):
N/A
Deploy preview for *privacytools-io* ready!
Built with commit 4467797f8e18643029d85eed20501ad375969e63
https://deploy-preview-1824--privacytools-io.netlify.com
@danarel no, not for consistency...... although I’d probably actually prefer adding links to the other two instead of removing these. Makes sense to list them IMO.
@danarel no, not for consistency...... although I’d probably actually prefer adding links to the other two instead of removing these. Makes sense to list them IMO.
For Proton and IVPN we only mention they have mobile clients.
I think we should either add them, or remove the links in the Mullvad description.
> Uhhh reviewing on my phone, but LGTM
For Proton and IVPN we only mention they have mobile clients.
I think we should either add them, or remove the links in the Mullvad description.
For Proton and IVPN we only mention they have mobile clients.
I think we should either add them, or remove the links in the Mullvad description.
I will add them and update this pull request @JonahAragon
> > Uhhh reviewing on my phone, but LGTM
>
> For Proton and IVPN we only mention they have mobile clients.
>
> I think we should either add them, or remove the links in the Mullvad description.
I will add them and update this pull request @JonahAragon
@danarel no, not for consistency...... although I’d probably actually prefer adding links to the other two instead of removing these. Makes sense to list them IMO.
Agreed. I rather add links then remove them.
> @danarel no, not for consistency...... although I’d probably actually prefer adding links to the other two instead of removing these. Makes sense to list them IMO.
Agreed. I rather add links then remove them.
@danarel @JonahAragon Updated the PR with the IVPN and ProtonVPN links.
[A preview can be found at that same netlify link](https://eager-colden-4cbe67.netlify.com/providers/vpn/)
jonah
approved these changes 2020-04-11 22:55:45 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
jonah
Description
Per the most recent activity on the old PR #1528 I have reconfigured this request to make the required changes.
Given that Mullvad now has published Apps on both main app stores for Android and iOS, I have included links and corrected the incorrect documentation that mobile clients were still in the works.
Resolves: #1528 which was erroneously closed. (By me, oops).
Check List
I understand that by not opening an issue about a software/service/similar addition/removal, this pull request will be closed without merging.
I have read and understand the contributing guidelines.
The project is Free Libre and/or Open Source Software
Netlify preview for the mainly edited page:
Netlify Preview here
Code repository of the project (if applicable):
N/A
Deploy preview for privacytools-io ready!
Built with commit
4467797f8ehttps://deploy-preview-1824--privacytools-io.netlify.com
@Mikaela @danarel @JonahAragon Pinging you to this, updating #1528. Apologies for the poor practice in opening a new PR.
Do we want to link to the clients since we don’t for the other providers?
Uhhh reviewing on my phone, but LGTM
@danarel no, not for consistency...... although I’d probably actually prefer adding links to the other two instead of removing these. Makes sense to list them IMO.
For Proton and IVPN we only mention they have mobile clients.
I think we should either add them, or remove the links in the Mullvad description.
I will add them and update this pull request @JonahAragon
Agreed. I rather add links then remove them.
@danarel @JonahAragon Updated the PR with the IVPN and ProtonVPN links.
A preview can be found at that same netlify link
👌
Looks good to me!
LGTM