Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

reset params before each route match #1

Open
wants to merge 1 commit into
base: test/continuous-integration
Choose a base branch
from

Conversation

olivM
Copy link

@olivM olivM commented Jun 5, 2013

this fix a bug referred by numerous people ( mtrpcic#57 )

as you know the maintainer of the original repo doesn't answer to pull request.

i hope you'll do !

( also you should add a package.json )

@igorlima
Copy link
Owner

igorlima commented Jun 6, 2013

Hi @olivM, our build failed on Travis CI, I'll take a look on it and then merge your changes in a different branch.

I don't think if it's good we maintain a different repo. I'll try to contact @mtrpcic and ask him for I be a contribuitor on his repo.

@purtuga
Copy link

purtuga commented Sep 18, 2013

Did you have any luck contacting @mtrpcic ? I was looking start using pathsj instead of sammy.js, but it looks like the project (pathjs) went cold...

@olivM
Copy link
Author

olivM commented Sep 19, 2013

no news ( but i didn't try for a while )

On Wednesday, September 18, 2013 at 11:20 PM, Paul T. wrote:

Did you have any luck contacting @mtrpcic (https://github.com/mtrpcic) ? I was looking start using pathsj instead of sammy.js, but it looks like the project (pathjs) went cold...


Reply to this email directly or view it on GitHub (#1 (comment)).

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

Successfully merging this pull request may close these issues.

3 participants