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
More information for the custom backend #92
Conversation
This link provides more information on how to test your regex
@@ -82,6 +82,7 @@ def test_add_distro(self): | |||
sess['email'] = 'pingou@pingoured.fr' | |||
|
|||
output = c.get('/distro/add', follow_redirects=True) | |||
print output.data |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this something left over from testing, or should this now always be in the tests?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left over from testing indeed, but not a problem when ran with runtests.sh
(cuts stdout unless the test breaks)
$('#example_txt').append( | ||
'<br /> More information in the ' | ||
+ '<a href="{{ url_for("about") }}#test-you-regex">' | ||
+ 'about#test-you-regex</a>' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe also make this use url_for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought you previously displayed the full URL. Never mind the comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did, but url_for will not show the full url, it will just give /about, which I thought was even worse than just about#...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, that's true.
…ive more information
This will make the page lighter and thus easier to load but also easier to read when looking at the sources
👍 |
Thanks ! |
More information for the custom backend
Fixes #71