Add Tor URL to VPN Provider website column #1072

jonah · 2019-08-01T05:35:01Z

nitrohorse commented

2019-08-01 05:35:01 +00:00

(Migrated from github.com)

Description

Resolves: https://github.com/privacytoolsIO/privacytools.io/issues/1058

Updated the "Website" column for both Email and VPN providers to not be sortable (doesn't make sense?) and to say "Website" for consistency vs "URL" like it was for the Email table.
Added Mullvad's Tor link. Not sure if I'm missing others?

## Description Resolves: https://github.com/privacytoolsIO/privacytools.io/issues/1058 - Updated the "Website" column for both Email and VPN providers to not be sortable (doesn't make sense?) and to say "Website" for consistency vs "URL" like it was for the Email table. - Added Mullvad's Tor link. Not sure if I'm missing others?

netlify[bot] commented

2019-08-01 05:35:39 +00:00

(Migrated from github.com)

Deploy preview for privacytools-io ready!

Built with commit 41d4c469b1

https://deploy-preview-1072--privacytools-io.netlify.com

Deploy preview for *privacytools-io* ready! Built with commit 41d4c469b16f05582b1d8d110b9b68e0f968ceb5 https://deploy-preview-1072--privacytools-io.netlify.com

Mikaela (Migrated from github.com) approved these changes 2019-08-01 12:08:29 +00:00

Mikaela (Migrated from github.com) left a comment

I don't understand all the changes, but https://deploy-preview-1072--privacytools-io.netlify.com/providers/vpn/ looks good to me.

nitrohorse commented

2019-08-01 17:39:19 +00:00

(Migrated from github.com)

TODO: validate this also works for the Classic view.

[edit] done, looks fine.

TODO: validate this also works for the Classic view. [edit] done, looks fine.

jonah approved these changes 2019-08-02 02:08:28 +00:00

jonah left a comment

LGTM. Thanks!

This repo is archived. You cannot comment on pull requests.

1 Participants

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: privacyguides/privacytools.io#1072