Improves the report issue button

This commit is contained in:
Cyberboss
2017-02-21 10:47:53 -05:00
parent e026ee3362
commit 550964e152
2 changed files with 5 additions and 9 deletions

View File

@@ -60,10 +60,10 @@ var/global/datum/getrev/revdata = new()
log_world("PR details successfully downloaded")
has_pr_details = TRUE
/datum/getrev/proc/GetTestMergeInfo()
/datum/getrev/proc/GetTestMergeInfo(header = TRUE)
if(!testmerge.len)
return ""
. = "The following pull requests are currently test merged:<br>"
. = header ? "The following pull requests are currently test merged:<br>" : ""
for(var/line in testmerge)
var/details = ""
if(has_pr_details)

View File

@@ -53,13 +53,9 @@
set hidden = 1
if(config.githuburl)
var/message = "This will open the Github issue reporter in your browser. Are you sure?"
var/first = TRUE
for(var/line in revdata.testmerge)
if(line)
if(first)
first = FALSE
message += ". The following experimental changes are active and are probably the cause of any new or sudden issues you may experience. If possible, please try to find a specific thread for your issue instead of posting to the general issue tracker:"
message += " <a href='[config.githuburl]/pull/[line]'>#[line]</a>"
if(revdata.testmerge.len)
message += "<br>The following experimental changes are active and are probably the cause of any new or sudden issues you may experience. If possible, please try to find a specific thread for your issue instead of posting to the general issue tracker:<br>"
message += revdata.GetTestMergeInfo(FALSE)
if(tgalert(src, message, "Report Issue","Yes","No")=="No")
return
src << link("[config.githuburl]/issues/new")