Current coverage is 47.72% (diff: 100%)

Merging #2480 into develop will not change coverage

@@            develop      #2480   diff @@
==========================================
  Files           191        191          
  Lines         16181      16181          
  Methods        2800       2800          
  Messages          0          0          
  Branches       2743       2743          
==========================================
  Hits           7722       7722          
  Misses         8459       8459          
  Partials          0          0          

Powered by Codecov. Last update 91bddcb...0316982

Couple of questions:

• do we actually need a field name? I remember this was inserted before 'just because' it was needed in the regex route thing.
• do we need get_redirect_route(), or is it sufficient to just add /? at the end of every regex?

@seanlip Do you mean that we add '/?' at the end of each regex then form a tuple, (regex_path, handler_class), just like the mapreduce_handlers?

@seanlip I tried the above proposal. But for create, either r'%s/<collection_id>' or r'%s/<collection_id>/?' doesn't work, no matter I visit /create/$collection_id or /create/$collection_id/.

@seanlip I tried with SimpleRoute and Route, but neither works. For SimpleRoute, it can't parse the arguments from the path. For Route, it will escape the '/?' I add as '/?'. Should we stick to the RedirectRoute thing?

OK, thanks for investigating! Yep, the current solution looks good; LGTM. Thanks!