Suggest wiping your environment to folks with bad build outcomes. #3347
Conversation
This is a basic start to trying to be more helpful to users.
|
I like the idea of giving some tips/advice on the "why this build has failed?". I would like to start testing something simple like this, how it goes, and maybe later try to make it "more intelligent" but I wouldn't spend more time on the intelligence at the moment but maybe more tips. |
|
Aye. I think we could build some nicer styles, and have easier ways for users to contribute them (eg. a YAML file of possible errors). But this is a nice start. |
| {% block title %}Wipe the build directories for a version {% endblock %} | ||
|
|
||
| {% block content %} | ||
|
|
||
| {% blocktrans with slug=version.slug %} | ||
| <h3>Remove build environment for <em>{{ slug }}</em> version.</h3> |
There was a problem hiding this comment.
I'd remove the h3 at least from the blocktrans
|
|
||
| <form method="post" action="#"> | ||
| <input type="submit" value="Wipe version"> | ||
| <input type="submit" value="Wipe {{ version.slug }}"> |
| {% url 'projects_advanced' build.version.project.slug as advanced_url %} | ||
| {% blocktrans %} | ||
| Don't want <pre>setup.py install</pre> called? | ||
| Change the <strong>Install Project</strong> setting in your <a href="{{ advanced_url }}">advanced settings</a>. |
There was a problem hiding this comment.
advanced_url should be in the blocktrans variable list
| {% url 'wipe_version' build.version.project.slug build.version.slug as wipe_url %} | ||
| {% blocktrans %} | ||
| Having trouble with your build environment? | ||
| Try <a href="{{ wipe_url }}">reseting it</a>. |
There was a problem hiding this comment.
wipe_url should be in the blocktrans variable list
There was a problem hiding this comment.
This is how the docs say to do it. https://docs.djangoproject.com/en/1.9/topics/i18n/translation/#blocktrans-template-tag
| {% if deleted %} | ||
| Your project environment has been wiped.<br><br> | ||
| {% blocktrans %} | ||
| Your project environment has been wiped.<br><br> |
There was a problem hiding this comment.
All <br> can be removed. Spacing should be forced with a surrounding p element, or other css.
agjohnson
added
the
PR: work in progress
This is a basic start to trying to be more helpful to users.