Merge lp://staging/~mvo/software-center/make-currency-part-of-the-db into lp://staging/software-center
Proposed by
Michael Vogt
Status: | Merged |
---|---|
Merged at revision: | 3179 |
Proposed branch: | lp://staging/~mvo/software-center/make-currency-part-of-the-db |
Merge into: | lp://staging/software-center |
Diff against target: |
75 lines (+14/-4) 4 files modified
softwarecenter/db/application.py (+2/-4) softwarecenter/db/update.py (+4/-0) softwarecenter/enums.py (+2/-0) tests/test_database.py (+6/-0) |
To merge this branch: | bzr merge lp://staging/~mvo/software-center/make-currency-part-of-the-db |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Gary Lasker (community) | Approve | ||
Paweł Stołowski (community) | Needs Fixing | ||
Review via email: mp+124367@code.staging.launchpad.net |
Description of the change
The unity team asked for a currency field in the DB so that it does
not have to be hardcoded in two places. Makes sense and prepares us
for future support of multiple currencies
To post a comment you must log in.
According to designs, currency should only include the symbol, e.g. '$', not 'USD $'. This also applies to the formatted_price returned by data-provider.