Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

Relax open work requirement #4221

Merged
merged 5 commits into from
Dec 12, 2016
Merged

Conversation

chadwhitacre
Copy link
Contributor

@chadwhitacre chadwhitacre commented Dec 2, 2016

Ever since the Gratipocalypse, we've required the "Teams" receiving money on Gratipay to go through an application and approval process. One of our main criteria for approval has been that Teams provide what we called "open work":

Open Work means that the Team provides a clear path for any individual to voluntarily begin contributing to the Team's work and to share in any revenue the Team generates.

Experience has shown this to be too restrictive. We still want to encourage and emphasize open work on Gratipay, but we don't want to require it. The purpose of this project is to relax the open work requirement.


✈️ This is the flight deck for the Relax Open Work Requirement project. ✈️


Todo

@chadwhitacre
Copy link
Contributor Author

Looks like we have two big pieces of work remaining before we can merge this branch to master: #4130 and #4157.

We should also figure out testing. Oooo! I have an idea ... ;-)

@chadwhitacre
Copy link
Contributor Author

Yesssss ... now to make sure that propagates to PRs against this branch ...

@chadwhitacre
Copy link
Contributor Author

👍

Seeing Travis on #4130 ...

screen shot 2016-12-06 at 8 39 06 am

@chadwhitacre
Copy link
Contributor Author

Cool. We'll just need to remove this branch from .travis.yml as part of our final merge to master.

This was referenced Dec 6, 2016
@chadwhitacre
Copy link
Contributor Author

chadwhitacre commented Dec 6, 2016

New plan: build all branches after all, so that Travis will test PR chains (e.g.).

@chadwhitacre
Copy link
Contributor Author

Rebased on master, was c2644b5.

@chadwhitacre chadwhitacre force-pushed the relax-open-work-requirement branch 2 times, most recently from 60fa606 to 1619f84 Compare December 7, 2016 19:59
@chadwhitacre
Copy link
Contributor Author

I squashed all of #4117 into a single commit, to make this integration PR easier to read and work with.

@chadwhitacre chadwhitacre force-pushed the relax-open-work-requirement branch from 1619f84 to cf13cc6 Compare December 7, 2016 20:03
@chadwhitacre
Copy link
Contributor Author

Blech. cf13cc6 is hitting the #4224 bug again, even with Faker 0.7.3. 😞

https://travis-ci.org/gratipay/gratipay.com/builds/182073772

@chadwhitacre chadwhitacre mentioned this pull request Dec 7, 2016
@chadwhitacre
Copy link
Contributor Author

Lol ... and 25018df hits the "13 failed" that we're also seeing at #4130 (comment). That one at least goes away with a rerun.

@chadwhitacre
Copy link
Contributor Author

That one at least goes away with a rerun.

Which of course makes it harder to debug. :-/

mattbk and others added 3 commits December 8, 2016 08:32
This means we'll have duplicate checks again, but that's a small price
to pay for having Travis run PR inception for us.
@chadwhitacre chadwhitacre force-pushed the relax-open-work-requirement branch from 25018df to ab63da0 Compare December 8, 2016 13:33
@chadwhitacre
Copy link
Contributor Author

Rebased, was 25018df.

@chadwhitacre
Copy link
Contributor Author

@mattbk Reverting is different than rebasing. Reverting adds a second commit that is the inverse of the first. Rebasing lets you remove the first.

@chadwhitacre
Copy link
Contributor Author

slack

@mattbk mattbk force-pushed the relax-open-work-requirement branch from fd2c60a to 698712d Compare December 9, 2016 22:03
@mattbk
Copy link
Contributor

mattbk commented Dec 9, 2016

Rebased, was fd2c60a.

Merge project-about into relax-open-work-requirement.
@chadwhitacre
Copy link
Contributor Author

I'm drafting a blog post.

@chadwhitacre
Copy link
Contributor Author

We need to figure out some sort of on-site notice, and maybe email.

@chadwhitacre
Copy link
Contributor Author

Simple post: https://gratipay.news/from-teams-to-projects-45c46718507b.

@chadwhitacre
Copy link
Contributor Author

Slight revision on the post: https://gratipay.news/from-teams-to-projects-45c46718507b.

@chadwhitacre
Copy link
Contributor Author

Driving in slack.

@chadwhitacre chadwhitacre merged commit dbaee7c into master Dec 12, 2016
@chadwhitacre
Copy link
Contributor Author

💃

@chadwhitacre chadwhitacre deleted the relax-open-work-requirement branch December 12, 2016 23:13
@chadwhitacre
Copy link
Contributor Author

chadwhitacre commented Dec 12, 2016

We're live with the UI and repo changes and blog post! Waiting to announce until tomorrow.


screen shot 2016-12-12 at 6 14 08 pm


screen shot 2016-12-12 at 6 13 13 pm

@chadwhitacre
Copy link
Contributor Author

Regression:

screen shot 2016-12-12 at 6 20 11 pm

@chadwhitacre
Copy link
Contributor Author

screen shot 2016-12-12 at 6 23 00 pm

@chadwhitacre
Copy link
Contributor Author

Regression fixed in #4230.

@chadwhitacre
Copy link
Contributor Author

@chadwhitacre
Copy link
Contributor Author

Alright ... email! Where did we do that before? How did we do it?

@chadwhitacre
Copy link
Contributor Author

Me starts with gratipay/inside.gratipay.com#376 ...

@chadwhitacre
Copy link
Contributor Author

Here we go:

gratipay/inside.gratipay.com#235
#3385 (comment)
#3510

I'm planning to write a script to queue this locally, with a production Mandrill key in local.env for sending.

@chadwhitacre
Copy link
Contributor Author

Yeah, that's got the whole live-blog on it.

@chadwhitacre
Copy link
Contributor Author

PR in #4232.

@chadwhitacre
Copy link
Contributor Author

Congratulations, people! With gratipay/inside.gratipay.com#972, we have relaxed our open work requirement! 🌝 👏 💃 🌻 🎆

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants