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

891546 build teaching kit thimble #275

Closed
wants to merge 40 commits into from
Closed

891546 build teaching kit thimble #275

wants to merge 40 commits into from

Conversation

obsoke
Copy link
Contributor

@obsoke obsoke commented Jul 18, 2013

DO NOT MERGE, THESE CHANGES WILL LIVE IN A THIMBLE PROJECT, NOT WM.ORG

@@ -0,0 +1,52 @@
# Copy this file to .env or just set these environment variables
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ignore this file, added to repo so I can deploy site on Heroku.

Copy link
Contributor

Choose a reason for hiding this comment

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

Remove for when we land this please

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, we should bring this up in our review/deployment strategy

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Bring up wut, having a fallback solution for non-existent .env files so we can deploy to Heroku? Agreed!

<head>
<meta charset="utf-8">
<title>Teaching Kit Template</title>
<link href='http://fonts.googleapis.com/css?family=Open+Sans:300,400,600' rel='stylesheet' type='text/css'>
Copy link
Contributor

Choose a reason for hiding this comment

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

Double quotes in HTML

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, is there an https resource available? If not, can we use our own hosted version?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There is an HTTPS version but you need to explicitly request it rather than using a protocol-less URL.

@obsoke
Copy link
Contributor Author

obsoke commented Sep 23, 2013

Old; closing.

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.

None yet

2 participants