search

cnodejs / conf

hu.js
4 stars 3 forks source link

Important fix to get deployed. This redirects based on the broken url we have in the existing sponsorship doc. #8

Closed glennblock closed 12 years ago

thoward commented 12 years ago

Fixed syntax and deployed.

On Tue, Aug 14, 2012 at 8:40 AM, Glenn Block notifications@github.comwrote:

You can merge this Pull Request by running:

git pull https://github.com/glennblock/conf master

Or view, comment on, or merge it at:

https://github.com/cnodejs/conf/pull/8 Commit Summary

  • adding redirect route to fix typo in the PDF link
  • added redirect for broken link in the PDF

File Changes

  • M app.js (6)

Patch Links

glennblock commented 12 years ago

Sorry on the typo Being brutally honest I didn't test it, shame on me :-(

On Wed, Aug 15, 2012 at 3:24 AM, Troy Howard notifications@github.comwrote:

Fixed syntax and deployed.

On Tue, Aug 14, 2012 at 8:40 AM, Glenn Block notifications@github.comwrote:

You can merge this Pull Request by running:

git pull https://github.com/glennblock/conf master

Or view, comment on, or merge it at:

https://github.com/cnodejs/conf/pull/8 Commit Summary

  • adding redirect route to fix typo in the PDF link
  • added redirect for broken link in the PDF

File Changes

  • M app.js (6)

Patch Links

— Reply to this email directly or view it on GitHub< https://github.com/cnodejs/conf/pull/8>.

— Reply to this email directly or view it on GitHubhttps://github.com/cnodejs/conf/pull/8#issuecomment-7737152.