Fixes #1643 - add colon and new line #1653
Conversation
tests/test_form.py
Outdated
| @@ -113,8 +113,10 @@ def test_build_formdata(self): | |||
| """The data body sent to GitHup API.""" | |||
| form_object = {'foo': 'bar'} | |||
| actual = form.build_formdata(form_object) | |||
|
|
|||
| print(actual) | |||
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
webcompat/form.py
Outdated
| @@ -235,7 +236,7 @@ def build_formdata(form_object): | |||
| **URL**: {url} | |||
| **Problem type**: {problem_type} | |||
| **Description**: {description} | |||
| **Steps to Reproduce** {steps_reproduce} | |||
| **Steps to Reproduce**: {steps_reproduce} | |||
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
zoepage
force-pushed
the
issues/1643/1
branch
2 times, most recently
from
f7e42f2 to
3dbd399
Compare
|
r? @miketaylr |
webcompat/form.py
Outdated
| @@ -59,7 +59,7 @@ | |||
| url_label = u'Site URL <span class="wc-Form-required">*</span>' | |||
| browser_test_label = u'Did you test in another browser?' | |||
|
|
|||
| steps_default = u"""For example, | |||
| steps_default = u"""\nFor example, | |||
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
webcompat/form.py
Outdated
|
|
||
| **Steps to Reproduce**: {steps_reproduce} |
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
zoepage
force-pushed
the
issues/1643/1
branch
3 times, most recently
from
720163d to
e1d22e3
Compare
There was a problem hiding this comment.
Almost there! :)
We still want STR to be the last items in the report, with browser/version, os, tested another browser above it.
The reason being there may be a large screenshot and it's nicer if all the metadata is up top.
(see #1653 (comment))
|
(basically we want the result to be just like 1683384, with the |
|
FINAL_FINAL_FINAL_REALLY_FINAL_THIS_TIME_VERSION-2.py @miketaylr |
yessss |
No description provided.