Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Revised "Hire 18F" content #2107

Merged
merged 4 commits into from
Dec 8, 2016
Merged

Revised "Hire 18F" content #2107

merged 4 commits into from
Dec 8, 2016

Conversation

coreycaitlin
Copy link

Fixes the hire page part of #2067

CircleCI

😎 PREVIEW

Changes proposed in this pull request:

  • Edit the content to focus on the essentials, with a more welcoming tone
  • Tell people who they're contacting — in this case, @d-kenny
  • Include more default behavior in the mailto link, including subject and a template for body text
  • Add a button

/cc @gemfarmer @gboone

@awfrancisco
Copy link

Looks good to me. And Dan's good with having his name on the site?

gboone
gboone approved these changes Dec 8, 2016
View reviewed changes
Copy link
Contributor

@gboone gboone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@coreycaitlin
Copy link
Author

@awfrancisco Yep! checked with him yesterday.

@gboone gboone merged commit 74af00a into dev Dec 8, 2016
@gboone gboone deleted the hire-rewrite branch December 8, 2016 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gboone gboone gboone approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@coreycaitlin @awfrancisco @gboone