#6759 GC import: 404 on ticket, list wiki pages, minor form tweaks

v1.1.0
closed
Cory Johns
import (116) google-code (27) sf-2 (994)
General
Dave Brondsema
2015-08-20
2013-10-15
Dave Brondsema
No
  • drop "at this time" from Google Code notes; it implicitly promises that those features are being worked on. In fact, better to switch the tone and say what can be imported (i.e. all public tickets) and for things that we don't handle (e.g. home page content, additional repos), tell them how to make it happen themselves. Figure out a way to add a link to site-specific docs.
  • if no default wiki page from GC, set it to browse_pages. Need to make that not show up on the list of pages though. Delete the Home page.
  • allow full GC url to be used in project name field, parse out the project name
  • error when reaching http://code.google.com/p/gmapcatcher/issues/detail?id=373 see trace below:

.

Traceback (most recent call last):
  File "/var/local/allura/ForgeImporters/forgeimporters/base.py", line 119, in import_tool
    mount_point=mount_point, mount_label=mount_label, **kw)
  File "/var/local/allura/ForgeImporters/forgeimporters/google/tracker.py", line 124, in import_tool
    for ticket_num, issue in GoogleCodeProjectExtractor.iter_issues(project_name):
  File "/var/local/allura/ForgeImporters/forgeimporters/google/__init__.py", line 138, in iter_issues
    yield (int(issue_id), cls(project_name, 'issue', issue_id=issue_id))
  File "/var/local/allura/ForgeImporters/forgeimporters/base.py", line 139, in __init__
    self.get_page(page_name, **kw)
  File "/var/local/allura/ForgeImporters/forgeimporters/base.py", line 172, in get_page
    parser(self.urlopen(self.url))
  File "/var/local/allura/ForgeImporters/forgeimporters/base.py", line 145, in urlopen
    return h.urlopen(req, retries=retries, codes=codes)
  File "/var/local/allura/Allura/allura/lib/helpers.py", line 975, in urlopen
    raise e
HTTPError: HTTP Error 404: Not Found

Discussion

  • Dave Brondsema

    Dave Brondsema - 2013-10-15
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -1,4 +1,4 @@
-* drop "at this time" from Google Code notes; it implicitly promises that those features are being worked on
+* drop "at this time" from Google Code notes; it implicitly promises that those features are being worked on.  In fact, better to switch the tone and say what can be imported (i.e. all public tickets) and for things that we don't handle (e.g. home page content, additional repos), tell them how to make it happen themselves.  Figure out a way to add a link to site-specific docs.
 * if no default wiki page from GC, set it to `browse_pages`.  Need to make that not show up on the list of pages though.  Delete the `Home` page.
 * allow full GC url to be used in project name field, parse out the project name
 * error when reaching http://code.google.com/p/gmapcatcher/issues/detail?id=373 see trace below:
     

  • Cory Johns - 2013-10-15
    • status: open --> in-progress
    • assigned_to: Cory Johns
     

  • Cory Johns - 2013-10-16
    • status: in-progress --> code-review
     

  • Cory Johns - 2013-10-16

    Bug fixes and tweaks, excluding the notes changes, are ready for review on:
    allura:cj/6759
    googlecodewikiimporter:cj/6759
    forge-classic:cj/6759

     

  • Dave Brondsema

    Dave Brondsema - 2013-10-16
    • QA: Dave Brondsema
     

  • Dave Brondsema

    Dave Brondsema - 2013-10-16

    Merged all of those.

     

  • Cory Johns - 2013-10-16
    • status: code-review --> in-progress
     

  • Cory Johns - 2013-10-16

    I'm not seeing the forge-classic branch having been merged. It's not critical, but I assume that's an oversight and it should be merged?

     

  • Cory Johns - 2013-10-16
    • status: in-progress --> code-review
     

  • Cory Johns - 2013-10-16

    Doc changes in:
    allura:cj/6759
    forge-classic:cj/6759

    Will need a INI change to make the link show, but can exist without it until the docs are ready.

     

  • Tim Van Steenburgh - 2013-10-17
    • status: code-review --> closed
     

  • Tim Van Steenburgh - 2013-10-17
    • Size: --> 2
     

Log in to post a comment.