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

Broken link for Sponsorship "Email us" due to missing `mailto' URI scheme in href. #1433

Closed
omsai opened this issue Sep 27, 2014 · 5 comments

Comments

@omsai
Copy link

omsai commented Sep 27, 2014

The "Email us" link in http://campus.openhatch.org/#sponsorship is broken in Mozilla Firefox 24.8.0. The link is rendered as http://campus.openhatch.org/hello@openhatch.org because of the missing mailto URI scheme. Said another way, Firefox renders <a href="some_email_address">foo</a> as http://myurl.com/some_email_address and thus needs the mailto: prefix, as in <a href="mailto:some_email_address">foo</a>.

Steps to reproduce:

  1. Go to http://campus.openhatch.org/#sponsorship
  2. Under "$X – Custom Level" click on the "Email us" link.

Result:

404 error as the web browser goes to http://campus.openhatch.org/hello@openhatch.org

Expected result:

Link should work like "get in touch" link in Sponsorship section.

@lewyckoff
Copy link

Are the campus.openhatch.org files in a separate git repo?

@shaunagm
Copy link
Member

lewyckoff added a commit to lewyckoff/oh-campus-foss that referenced this issue Oct 15, 2014
@lewyckoff
Copy link

Is this commit ok?

@omsai
Copy link
Author

omsai commented Oct 17, 2014

It looks great to me. Thanks for contributing. You can go to your repository fork and submit a pull request.

ehashman added a commit to openhatch/oh-campus-foss that referenced this issue Nov 2, 2014
@ehashman
Copy link
Member

ehashman commented Nov 2, 2014

Merged openhatch/oh-campus-foss#16 so I'm going to close this.

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

No branches or pull requests

4 participants