Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix/scoped slug regeneration #513

Merged
merged 3 commits into from
Jan 28, 2014

Conversation

PikachuEXE
Copy link
Contributor

Fix for #512

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 547ae17 on PikachuEXE:fix/scoped-slug-regeneration into * on norman:master*.

@glebm
Copy link

glebm commented Jan 15, 2014

This indeed fixes #515. Thanks Pikachu!

@PikachuEXE
Copy link
Contributor Author

I shall add a test for non scope case

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling b5bf8f7 on PikachuEXE:fix/scoped-slug-regeneration into * on norman:master*.

@PikachuEXE
Copy link
Contributor Author

Should I ping someone...?

@parndt
Copy link
Collaborator

parndt commented Jan 28, 2014

Looks good to me; @norman ?

@norman
Copy link
Owner

norman commented Jan 28, 2014

Looks good! Thanks for this, and sorry for the slow response.

norman added a commit that referenced this pull request Jan 28, 2014
@norman norman merged commit cc1e1bc into norman:master Jan 28, 2014
@PikachuEXE
Copy link
Contributor Author

@norman You forgot to mention this PR in changelog :)

@norman
Copy link
Owner

norman commented Feb 26, 2014

Oops, sorry. Feel free to add it and send a PR.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants