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

Improve project application #4271

Merged
merged 8 commits into from
Jan 16, 2017
Merged

Improve project application #4271

merged 8 commits into from
Jan 16, 2017

Conversation

mattbk
Copy link
Contributor

@mattbk mattbk commented Jan 1, 2017

Simplifying this to just changing the button on the front page.

Including #4270, #4268 (comment), #3859, and onboarding in general.

@mattbk mattbk added this to the Tighten Onboarding Flow milestone Jan 5, 2017
@mattbk mattbk added the Review label Jan 8, 2017
@chadwhitacre
Copy link
Contributor

Kicked the other half of travis ...

@@ -82,7 +82,7 @@ page_id = "homepage"
{% endif %}

<form action="new" class="apply">
<button type="submit">{{ _("Apply to Join") }}</button>
<button type="submit">{{ _("Fund your Project") }}</button>
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: your -> Your

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed in e4605e6.

Copy link
Contributor

@nobodxbodon nobodxbodon left a comment

Choose a reason for hiding this comment

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

LGTM. Plz let me know if you are good to merge.

@mattbk
Copy link
Contributor Author

mattbk commented Jan 16, 2017

Go for it.

@nobodxbodon nobodxbodon merged commit 22d10e4 into master Jan 16, 2017
@nobodxbodon nobodxbodon deleted the improve-apply branch January 16, 2017 18:52
@mattbk
Copy link
Contributor Author

mattbk commented Jan 16, 2017

!m @nobodxbodon.

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

Successfully merging this pull request may close these issues.

3 participants