Archived
Clarify sponsorship policy #1508
Closed
jonah
wants to merge 3 commits from
sponsorship-edits-nov2019 into master
pull from: sponsorship-edits-nov2019
merge into: :master
:master
:dependabot/bundler/nokogiri-1.13.6
:dependabot/bundler/addressable-2.8.0
:freddy-m-patch-3
:pr-add_RemoveMyPhone_sponsor
:pr-browser_cleanup_1257_1328_1430
:freddy-m-patch-2
:freddy-m-patch-1
:pr-vpn_hated_one_video
:cdn
:update-nitrohorse-image
:promote-metager-to-card
:hardware
:pr-add_azirevpn
:pr-add_mailfence
:shop
:1673
:pr/1658
:i18n-simple
:sponsorship-edits-nov2019
:i18n
:ipfs
:blacklight447-ptio-patch-3
:blog
:remove-windows-icons
:pr/1147
:i18n-testing
:add-beautify
Dismiss Review
Are you sure you want to dismiss this review?
Labels
Clear labels
:mag:🤖 Search Engines
I2P
OS
Self-contained networks
Social media
Tor
WIP
XMPP
[m]
approved
dependencies
duplicate
feedback wanted
high priority
iOS
low priority
stale
streaming
todo
wontfix
₿ cryptocurrency
ℹ️ help wanted
↔️ file sharing
⚙️ web extensions
✨ enhancement
❌ software removal
🆕 software suggestion
🆘 critical
🇦🇶 translations
🌐 Social News Aggregators
🌐 hosting
🌐 website issue
🏠 housekeeping
🐛 bug
👁️ browsers
👥 team chat
💢 conflicting
💬 discussion
💻 hardware
📁 file storage
📝 correction
📧 email
🔎 research required
🔐 password managers
🔒 VPN
🔒 file encryption
🖊️ digital notebooks
🗄️ DNS
🗨️ instant messaging (im)
🚫 Windows
🤖 Android
🦊 Firefox
🧰 productivity tools
The Invisible Internet Project (I2P)
Operating Systems
Anything covering the Tor network
active work in progress, do not merge or PR (yet)!
Extensible Messaging and Presence Protocol
Matrix protocol
approved, waiting for a PR
Pull requests that update a dependency file
A label for stalebot if it gets added
Anything related to media streaming.
Issues or bugs that will not be fixed and/or do not have significant impact on the project.
Browser Extension related issues
Anything covering a translated version of the site
*Technical* issues with the website.
Anything primarily related to site cleanup.
Correction of content on the website
Virtual Private Network
Domain Name System
Firefox & forks, about:config etc.
No labels
Milestone
No items
No Milestone
No due date set.
Dependencies
No dependencies set.
Reference: privacyguides/privacytools.io#1508
Reference in New Issue
Block a user
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.
https://github.com/privacytoolsIO/privacytools.io/pull/1451#issuecomment-553954762
@davegson I would be interested in your thoughts.
https://deploy-preview-1508--privacytools-io.netlify.com/sponsors/
Deploy preview for privacytools-io ready!
Built with commit
72cd94438fhttps://deploy-preview-1508--privacytools-io.netlify.com
I really like the clarification! Draws a clear line to rule out shady actors but still gives space for you as a team to validate each request on a case by case basis 👍👍👍
Two sentences I feel you can improve:
It's odd to single out governments here. There are many different types of malicious entities ready to spoof user data. I'd rephrase it to something more generic like: "to prevent malicious actors from..."
I think there is a logical error in here, first I do not know if you can say: "approve/deny a sponsorship" Would it not rather be that you approve/deny a sponsorship request (or inquiry) [I'm not fluent]
Also, I'd split the sentence into two so the approve/deny part is in one sentence and the reason for denying in the latter. Otherwise the misalignment seems to be both the reason for the accept & deny.
Something like:
We reserve the right to approve or deny all sponsorship inquieries from receiving the benefits detailed below. We will deny any incoming requests if we believe the organization or product is not aligned with our key values.
Make it your own 😉
So the problem is that the sponsorship tiers are published on Open Collective and nothing technically is stopping somebody like say, Facebook, from giving us $250/month or whatever. So the main point is that if you don't first email sponsors@privacytools.io and get in touch, there's no guarantee that you'll get your logo on the website or something, simply because you sent money over.
Why the moving away from variable? I am not reading further while wondering this.
@@ -17,3 +20,4 @@/LICENSE.txt @JonahAragon/.travis.yml @JonahAragon @Shifterovich @nitrohorse/CNAME @JonahAragon/nginx/ @JonahAragonI am curious on why?
@@ -46,3 +46,3 @@<p><input type="text" value="{{ site.name }} - Encryption and tools to protect against global mass surveillance - {{ site.production_url }}" class="form-control input-lg onclick-select"><input type="text" value="PrivacyTools - Encryption and tools to protect against global mass surveillance - {{ site.production_url }}" class="form-control input-lg onclick-select"><span class="help-block">For easy copy and paste. Share this text snippet.</span>Why not using the variable?
@@ -13,3 +12,4 @@<input type="search" name="q" class="form-control input-lg" id="q" placeholder="Try PrivacyTools Search, a Privacy-Respecting Search Engine" autocomplete="off" value=""><span class="input-group-btn"><button type="submit" class="btn btn-primary input-lg"><span class="hide_if_nojs"><i class="fas fa-search"></i> </span><span class="hidden active_if_nojs">start search</span></button></span>I don't understand the variable change here either.
@@ -46,3 +46,3 @@<p><input type="text" value="{{ site.name }} - Encryption and tools to protect against global mass surveillance - {{ site.production_url }}" class="form-control input-lg onclick-select"><input type="text" value="PrivacyTools - Encryption and tools to protect against global mass surveillance - {{ site.production_url }}" class="form-control input-lg onclick-select"><span class="help-block">For easy copy and paste. Share this text snippet.</span>Since I'm already on this PR. I agree the why needs to be elaborated. Your commits messages do not elaborate on that.
Wouldn't iE @abbluiz need the
{{ site.name }}variable to more easily support the Portuguese version?@JonahAragon, I like the text tweaks. 👍
@@ -46,3 +46,3 @@<p><input type="text" value="{{ site.name }} - Encryption and tools to protect against global mass surveillance - {{ site.production_url }}" class="form-control input-lg onclick-select"><input type="text" value="PrivacyTools - Encryption and tools to protect against global mass surveillance - {{ site.production_url }}" class="form-control input-lg onclick-select"><span class="help-block">For easy copy and paste. Share this text snippet.</span>It doesn’t make sense to me as we shift to in-repo translations (#1509), and it was implemented to solve a problem that doesn’t currently exist.
Ok, but
@@ -11,2 +10,3 @@PrivacyTools is an unincorporated community developing this website and a number of privacy-friendly services. The current list of public team members [can be found on GitHub](https://github.com/orgs/privacytoolsIO/people). In order to operate these services, PrivacyTools receives hosting and administration services from Aragon Ventures LLC.## How does {{ site.name }} collect data about me?## How does PrivacyTools collect data about me?I wonder if https://www.privacytools.io/about/ would be a better or at least good to mention here?
Right or deny makes no sense to me, did you perhaps mean right to?