Skip to content

Commit

Permalink
Bugs: change bugzilla refs to github issues
Browse files Browse the repository at this point in the history
  • Loading branch information
dwaynebailey committed Oct 30, 2014
1 parent fc7981c commit 29fdd17
Show file tree
Hide file tree
Showing 7 changed files with 24 additions and 9 deletions.
4 changes: 2 additions & 2 deletions README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ several purposes.
References
----------
* Website: <http://translate.sourceforge.net/wiki/virtaal/index>
* Bug tracker: <http://bugs.locamotion.org/>
* Bug tracker: <https://github.com/translate/virtaal/issues>

Installation
------------
Expand All @@ -27,7 +27,7 @@ Reporting bugs
--------------
Please report bugs by clicking on '*Help->Report bug...*' from within Virtaal
(this requires a Bugzilla account). Alternatively report directly using this
URL: <http://bugs.locamotion.org/enter_bug.cgi?product=Virtaal>
URL: <https://github.com/translate/virtaal/issues/new>

Taking the time to create a bugzilla account will help ensure that you can track
progress, provide feedback and allows others to see your bug report.
Expand Down
4 changes: 2 additions & 2 deletions docs/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -279,8 +279,8 @@

extlinks = {
# :role: (URL, prefix)
'bug': ('http://bugs.locamotion.org/show_bug.cgi?id=%s',
'bug '),
'issue': ('https://github.com/translate/translate/issues/%s',
'issue '),
'man': ('http://linux.die.net/man/1/%s', ''),
'wp': ('http://en.wikipedia.org/wiki/%s', ''),
'wiki': ('http://translate.sourceforge.net/wiki/%s', ''),
Expand Down
2 changes: 1 addition & 1 deletion docs/development_plans.rst
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ Changes to Virtaal will be guided by these guidelines:
- Maximum functionality without setup and configuration

View a list of `open requests and bugs
<http://bugs.locamotion.org/buglist.cgi?query_format=specific&order=relevance+desc&bug_status=__open__&product=Virtaal&content=>`_.
<https://github.com/translate/virtaal/issues>`_.

Features that are likely to be developed in the foreseeable future:

Expand Down
2 changes: 1 addition & 1 deletion docs/index.rst
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ Installation
Contact
=======
- Chat in our IRC channel `#pootle <irc://freenode.net/#pootle>`_
- `Report bugs <http://bugs.locamotion.org/enter_bug.cgi?product=Virtaal>`_
- `Report bugs <https://github.com/translate/virtaal/issues/new>`_
- Join the `Translate-devel mailing list
<https://lists.sourceforge.net/lists/listinfo/translate-devel>`_

Expand Down
16 changes: 16 additions & 0 deletions docs/redirects.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
#!/bin/bash

extension=".rst"

product=$1
shift 1
files=$*

echo "# Automatically generated - please review each redirect"
for file in $files
do
base_filename=$(basename $file $extension)
wikifile=/wiki/$product/$base_filename
rtdfile=http://docs.translatehouse.org/projects/$product/en/latest/${base_filename}.html
printf "Redirect Permanent %-40s%s\n" ${wikifile} ${rtdfile}
done
3 changes: 1 addition & 2 deletions virtaal/views/mainview.py
Original file line number Diff line number Diff line change
Expand Up @@ -828,8 +828,7 @@ def _on_recent_file_activated(self, chooser):

def _on_report_bug(self, _widget=None):
from virtaal.support import openmailto
from virtaal import __version__
openmailto.open("http://bugs.locamotion.org/enter_bug.cgi?product=Virtaal&version=%s" % __version__.ver)
openmailto.open("https://github.com/translate/virtaal/issues/new")

This comment has been minimized.

Copy link
@friedelwolff

friedelwolff Mar 12, 2016

Member

Thanks for this, Dwayne.

Do you think we should encourage searching first? Pointing to /translate/virtaal/issues/ works for searching, but then a new issue is an extra click. On the other hand, the "new issue" URL only gives a login page for someone not logged in, which seems even less likely to result in a "conversion".

Another thing: we can prefill the form if we want, with something like
https://github.com/translate/virtaal/issues/new?title=foo&body=bar&labels[]=bug
I wouldn't want something in the title, but this way we can include __version__.ver as we did before, which is quite useful. I propose the label "inproduct" (which might help us see if this functionality is used much in the software). We might want to additionally include the OS, the language combination, UI language, and versions of important dependencies (python, toolkit, etc). Any others? We can rather put a few more, since it is easy for users to remove if they want. What do you think?


def _on_store_closed(self, store_controller):
for widget_name in ('mnu_saveas', 'mnu_close', 'mnu_update', 'mnu_properties', 'mnu_binary_export'):
Expand Down
2 changes: 1 addition & 1 deletion virtaal/views/storeview.py
Original file line number Diff line number Diff line change
Expand Up @@ -159,5 +159,5 @@ def _on_preview(self, menu_item):
def _on_style_set(self, widget, prev_style):
# The following color change is to reduce the flickering seen when
# changing units. It's not the perfect cure, but helps a lot.
# http://bugs.locamotion.org/show_bug.cgi?id=1412
# https://github.com/translate/virtaal/issues/1412
self._treeview.modify_base(gtk.STATE_ACTIVE, widget.style.bg[gtk.STATE_NORMAL])

0 comments on commit 29fdd17

Please sign in to comment.