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

revise the Team application #3694

Merged
merged 14 commits into from
Aug 21, 2015
Merged

revise the Team application #3694

merged 14 commits into from
Aug 21, 2015

Conversation

chadwhitacre
Copy link
Contributor

Closes #3677.

This is the capitalization we're using on the About pages.
In #3677 we decided not to use these questions anymore.
@chadwhitacre
Copy link
Contributor Author

Rebased on master.

@chadwhitacre
Copy link
Contributor Author

  • Add a couple more checkboxes for public review and payroll agreement.
  • Add validation for URLs.
  • Test with empty URLs (for pre-existing Teams)
  • i18n

@chadwhitacre
Copy link
Contributor Author

Right direction, @mattbk et al.?

screen shot 2015-08-19 at 9 02 33 pm
screen shot 2015-08-19 at 9 02 41 pm

@mattbk
Copy link
Contributor

mattbk commented Aug 20, 2015

I like it.

@chadwhitacre
Copy link
Contributor Author

I propose that we land this without "How to Review Team Applications," and then once we write that we link "publicly reviewed" there instead of to the applications issue tracker.

screen shot 2015-08-20 at 10 45 12 am

@chadwhitacre
Copy link
Contributor Author

Empty URLs are fine, we just don't show those links:

screen shot 2015-08-20 at 10 46 16 am

@chadwhitacre
Copy link
Contributor Author

Here's what it looks like with the links filled out:

screen shot 2015-08-20 at 10 50 13 am

@chadwhitacre
Copy link
Contributor Author

I keep waffling about the "Issues" term. Waddya think?

  • Issues
  • TODO
  • Tracker
  • Tickets
  • Work Available

@chadwhitacre
Copy link
Contributor Author

I like something more general, like TODO or Work Available, because it leaves open the specific tooling in use.

@mattbk
Copy link
Contributor

mattbk commented Aug 20, 2015

"Work Available" is clear.

@chadwhitacre
Copy link
Contributor Author

"To-do" is a pretty common term (I don't think it's limited to geeks, is it? TODO would be, but To-do I think is okay). And it's what I used for the field name, todo_url. :-)

@mattbk
Copy link
Contributor

mattbk commented Aug 20, 2015

I would avoid TODO because it looks like an acronym to non-coders.

@chadwhitacre
Copy link
Contributor Author

@mattbk How about if it's spelled "To-do" instead (see my previous comment)?

@chadwhitacre
Copy link
Contributor Author

Another thought: Teams are in between. On the one hand a Team page should appeal to ~users who want to give payments to the Team, and on the other to ~users who want to do work and take payroll. Out of scope for this PR but something to keep in mind when we're able to circle back around to #3248.

@chadwhitacre chadwhitacre mentioned this pull request Aug 20, 2015
@chadwhitacre
Copy link
Contributor Author

Okay! I think this PR is ready for final review. Deployment notes:

  • find URLs to backfill for existing teams

@chadwhitacre
Copy link
Contributor Author

@mattbk et al. Wanna help me fill out this CSV? :)

/me starts at the bottom ...

@mattbk
Copy link
Contributor

mattbk commented Aug 21, 2015

Can you give me access?

@chadwhitacre
Copy link
Contributor Author

@mattbk I would expect that you can simply click and start editing. No?

@chadwhitacre
Copy link
Contributor Author

I looked for a "sharing" setting and didn't see it ...

@mattbk
Copy link
Contributor

mattbk commented Aug 21, 2015

I get "Unauthorized [403]" and I am logged in with github.

@chadwhitacre
Copy link
Contributor Author

Interesting! Okay, one sec ...

@chadwhitacre
Copy link
Contributor Author

@chadwhitacre
Copy link
Contributor Author

"Demo expires in in 49 minutes." Hurry! 💃

@mattbk
Copy link
Contributor

mattbk commented Aug 21, 2015

I charged through a bunch of them, but have to run now.

@chadwhitacre
Copy link
Contributor Author

!m @mattbk

@chadwhitacre
Copy link
Contributor Author

@chadwhitacre
Copy link
Contributor Author

Hmmm ... I think we should keep the revenue_model, getting_involved, and getting_paid columns around, but only make them visible to the Team owner on the edit page (#3545), in case they want to copy/paste some of that text over to the main textarea.

@chadwhitacre
Copy link
Contributor Author

@chadwhitacre
Copy link
Contributor Author

I manually updated eight homepages that didn't have a protocol prefixed (one is a mailto:!).

@chadwhitacre
Copy link
Contributor Author

I added a little script to that gist, for generating SQL updates from the CSV.

@chadwhitacre
Copy link
Contributor Author

Alright. Going for it ...

chadwhitacre added a commit that referenced this pull request Aug 21, 2015
revise the Team application
@chadwhitacre chadwhitacre merged commit 6da0110 into master Aug 21, 2015
@chadwhitacre chadwhitacre deleted the team-app branch August 21, 2015 03:47
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