CSV Export On Dev & Prod Results In Error

VERIFIED FIXED

Status

Websites Graveyard
mobilepartners.mozilla.org
VERIFIED FIXED
3 years ago
10 months ago

People

(Reporter: bensternthal, Assigned: jgmize)

Tracking

Beta
x86
Mac OS X

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
If you click CSV export in dev or production you get an error page.
(Reporter)

Comment 1

3 years ago
Created attachment 8462677 [details]
Screen Shot 2014-07-25 at 10.15.04 AM.png

screenshot of error.. not much help but the URL is in there.
(Assignee)

Comment 2

3 years ago
Here is the traceback for the error:

Traceback (most recent call last):

  File "/home/fxoss/.virtualenvs/fxoss/local/lib/python2.7/site-packages/django/core/handlers/base.py", line 112, in get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)

  File "/home/fxoss/.virtualenvs/fxoss/project/protected_assets/views.py", line 130, in export_signedagreement_csv
    generate_row)

  File "/home/fxoss/.virtualenvs/fxoss/project/protected_assets/views.py", line 116, in export_csv
    writer.writerow(generate_row(sa))

UnicodeEncodeError: 'ascii' codec can't encode characters in position 0-4: ordinal not in range(128)
Assignee: mkelly → jmize
Status: NEW → ASSIGNED

Comment 3

3 years ago
Commits pushed to master at https://github.com/mozilla/fxoss

https://github.com/mozilla/fxoss/commit/183023a08727a1e44dcaf68ef1bd266930670d2e
Fix UnicodeEncodeError in csv export: Bug 1044094

https://github.com/mozilla/fxoss/commit/7cfd9c4f198fa1a732e1e1d1d06dcc7bd72cd2c9
Merge pull request #163 from jgmize/fix-csv-export-unicode-error

Fix UnicodeEncodeError in csv export: Bug 1044094
(Reporter)

Comment 4

3 years ago
I am pretty sure you resolved this, please mark a such if true.
Flags: needinfo?(jmize)
(Assignee)

Comment 5

3 years ago
This bug was fixed by the commit pushed to master noted in comment #3 by the github robot, and has been verified in dev, but we have not yet pushed this to prod. Marking as resolved fixed as the code has landed in master, and will verify immediately after next production push.
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Flags: needinfo?(jmize)
Resolution: --- → FIXED
(Reporter)

Updated

3 years ago
Status: RESOLVED → VERIFIED
Product: Websites → Websites Graveyard
You need to log in before you can comment on or make changes to this bug.