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

Replace contact email with the foundation's contact form. #152

Closed
wants to merge 1 commit into from

Conversation

mikeal
Copy link
Contributor

@mikeal mikeal commented Jul 17, 2015

This is the preferred method for sending membership leads to the foundation.

@indexzero
Copy link
Contributor

+1 Lets iframe like it's 2005.

@robertkowalski
Copy link
Contributor

LGTM

mikeal added a commit that referenced this pull request Jul 17, 2015
PR: #152
PR-URL: nodejs/nodejs.org#152
Reviewed-By: Robert Kowalski <rok@kowalski.gd>
Reviewed-By: Charlie Robbins
@robertkowalski
Copy link
Contributor

merged as 02f986f

@mikeal
Copy link
Contributor Author

mikeal commented Jul 17, 2015

Cool, how long does it take for this to go live on the website?

@robertkowalski
Copy link
Contributor

i just deployed it

@robertkowalski
Copy link
Contributor

but I get

Mixed Content: The page at 'https://nodejs.org/foundation/' was loaded over HTTPS, but requested an insecure resource 'http://go.linuxfoundation.org/l/6342/2015-05-15/2cnz97'. This request has been blocked; the content must be served over HTTPS.

@mikeal
Copy link
Contributor Author

mikeal commented Jul 17, 2015

ya, I see that. and that resource has a bad cert.

@robertkowalski
Copy link
Contributor

@mikeal fix on the way!

@robertkowalski
Copy link
Contributor

@robertkowalski
Copy link
Contributor

I also opened nodejs/nodejs.org#154 as a learning :)

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

Successfully merging this pull request may close these issues.

3 participants