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.

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

Hotfix/Contact form fix #1070

Merged
merged 6 commits into from
Jun 7, 2016
Merged

Hotfix/Contact form fix #1070

merged 6 commits into from
Jun 7, 2016

Conversation

frenchbread
Copy link
Contributor

@frenchbread frenchbread commented Jun 7, 2016

Closes #961

Fixes:

  • Contact form is hidden if the email address is not provided

@frenchbread
Copy link
Contributor Author

@apinf/developers Please review

@jykae jykae self-assigned this Jun 7, 2016
@jykae
Copy link
Contributor

jykae commented Jun 7, 2016

@frenchbread Cleaned up console logs, tested contact form for clean db and existing one after that. Verified to work on both and if contact form email does not exist, contact form is not showing.

Feeling good about this, merging in.

@jykae jykae merged commit 766dd8f into develop Jun 7, 2016
@jykae jykae deleted the hotfix/api-umbrella-methods-sync branch June 7, 2016 13:44
@jykae jykae removed the in progress label Jun 7, 2016
@frenchbread
Copy link
Contributor Author

@jykae Cool!

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

Successfully merging this pull request may close these issues.

None yet

3 participants