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

Uri adjust crashes on mailto: and doesn't interpolate form actions #159

Merged
merged 5 commits into from
Oct 5, 2016

Conversation

melmothx
Copy link
Contributor

@melmothx melmothx commented Oct 5, 2016

Provide tests for now.

Also fix an indirect call in T::F main class.

The first check the form action, the second the mailto: links

The first is not interpolated, the second triggers a crash
@melmothx
Copy link
Contributor Author

melmothx commented Oct 5, 2016

Looks fixed to me now.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 85.023% when pulling 3f95473 on melmothx:fix-mailto-uri-adjust into 5eac275 on racke:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 85.023% when pulling 3f95473 on melmothx:fix-mailto-uri-adjust into 5eac275 on racke:master.

@racke racke merged commit c1ecb91 into racke:master Oct 5, 2016
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