Ubuntu Apps Directory

Code review comment for lp:~canonical-isd-hackers/ubuntu-webcatalog/useragent_778562

  1. useragent_778562
  2. Merge into trunk
Revision history for this message
David Owen (dsowen) wrote :

I really like how you factored the tests!

Approved, with recommendation that Linux fields not be set in get_user_os() when not a Linux browser.

review: Approve

« Back to merge proposal