Rajeshucsb master #10

Merged
merged 23 commits into from Aug 19, 2012

3 participants

@technoweenie

Combine #7 and #8. Should renumber this to 15 :)

@travisbot

This pull request fails (merged 08d544a into a0639dd).

@technoweenie

I don't really like how the length/charset are passed in like this. I think we need to extract the some URL shortener classes:

  • The older md5 one
  • A length/charset one
  • One that encodes an autoincrement number into a string (seems to be a very common algorithm for shorteners).

This PR should be good for now though.

@travisbot

This pull request passes (merged 8b28a22 into a0639dd).

@technoweenie technoweenie merged commit 1ae168f into master Aug 19, 2012

1 check passed

Details default The Travis build passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment