Requested Changes

Author: bansalnitish

readthedocs/builds/views.py

@@ -11,6 +11,8 @@
 from django.contrib.auth.decorators import login_required
 from django.core.urlresolvers import reverse
 from django.utils.decorators import method_decorator
+from requests.utils import quote
+from future.backports.urllib.parse import urlparse
 
 from readthedocs.builds.models import Build, Version
 from readthedocs.core.utils import trigger_build
@@ -82,6 +84,28 @@ class BuildDetail(BuildBase, DetailView):
     def get_context_data(self, **kwargs):
         context = super(BuildDetail, self).get_context_data(**kwargs)
         context['project'] = self.project
+        scheme = ("https://github.com/rtfd/readthedocs.org/issues/new"
+                  "?title={title}{build_id}"
+                  "&body={body}")
+
+        body = ("# Details:\n\n"
+                "*Project URL: https://readthedocs.org/projects/{projname}/\n"
+                "*Build URL(if applicable): https://readthedocs.org{build_path}\n"
+                "*Read the Docs username(if applicable): {uname}\n\n"
+                "## Expected Result\n\n"
+                "*A description of what you wanted to happen*\n\n"
+                "## Actual Result\n\n"
+                "*A description of what actually happened*").format(
+            projname=self.project, build_path=self.request.path,
+            uname=self.request.user)
+
+        scheme_dict = {'title': quote("Build error with build id #"),
+                       'build_id': context['build'].id,
+                       'body': quote(body)}
+
+        issue_url = scheme.format(**scheme_dict)
+        issue_url = urlparse(issue_url).geturl()
+        context['issue_url'] = issue_url
         return context
 
 

readthedocs/templates/builds/build_detail.html

@@ -1,4 +1,3 @@
-
 {% extends "projects/base_project.html" %}
 
 {% load i18n %}
@@ -163,16 +162,11 @@ <h3>{% trans "Error" %}</h3>
           {{ build.error }}
         </p>
         <p>
-        {% blocktrans with build_id=build.pk proj_name=build.project.name build_path=request.get_full_path user_name=request.user %}
-         Report any build issues <a href="https://github.com/rtfd/readthedocs.org/issues/new?
-         title=Build%20error%20with%20build%20id%20%23{{ build_id }}
-         &body=%23%23%20Details%0A%0A
-         *%20Project%20Url%3A%20https://readthedocs.org/projects/{{ proj_name }}/%0A
-         *%20Build%20URL%20(if%20applicable)%3A%20https://readthedocs.org{{ build_path }}%0A
-         *%20Read%20the%20Docs%20username%20(if%20applicable)%3A%20{{ user_name }}%0A%0A
-         %23%23%20Expected%20Result%0A%0A*%20A%20description%20of%20what%20you%20wanted%20to%20happen*%0A%0A
-         %23%23%20Actual%20Result%0A%0A*A%20description%20of%20what%20actually%20happened*%0A">here</a>.
-        {% endblocktrans %}
+          {% block github_issue_link %}
+            {% blocktrans with url=issue_url %}
+              Report any build issues <a href="{{ url }}">here</a>.
+            {% endblocktrans %}
+          {% endblock %}
         </p>
       </div>
 
@@ -209,4 +203,4 @@ <h3>{% trans "Error" %}</h3>
     {% endif %}
 
   </div>
-{% endblock %}
+{% endblock %}