769249 - [API] CSV export

Status: VERIFIED FIXED

(Mozilla Reps Graveyard :: reps.mozilla.org, task)

Comment 1

[[github robot]]

Commit pushed to master at https://github.com/mozilla/remo

https://github.com/mozilla/remo/commit/be2ae0d8c1797f4cbd8ee9002a9b8dee5d2e077c
[fix bug 769249] Add API CSV serializer.

Comment 2

[Aditya Jalgaonkar [:Adi_J]]

Please provide verification steps for QA.
Else please tag the bug as [qa-]

Comment 3

[Giorgos Logiotatidis [:giorgos]]

Hey Aditya,

To verify please do:

1. Go to /people
2. Click 'advanced options' (the 'tools' icon)
3. Click on CSV export
4. A download prompt with a valid CSV file should appear

Thanks

Comment 4

[Aditya Jalgaonkar [:Adi_J]]

The CSV export title does not seem to be what it should be:

http://screencast.com/t/TrGpfh1Flp

Comment 5

[Giorgos Logiotatidis [:giorgos]]

Can you please explain? I don't get the error :S

Comment 6

[Aditya Jalgaonkar [:Adi_J]]

I mean the Title of the file if you see the screencast is something like "cUX2KqS7.csv"
Would it be possible not to give a random file name and generate a static name or Static name with date "Remo_People_Search<Date>.csv" Would give a better perspective on the file when downloaded.

Comment 7

[Giorgos Logiotatidis [:giorgos]]

Nice catch, not an easy fix now though. Could you please file an enhancement bug and verify this one?

Thanks!

Comment 8

[Aditya Jalgaonkar [:Adi_J]]

Sure thing :)
Filing a new bug for the enhancement.

Comment 9

[Aditya Jalgaonkar [:Adi_J]]

Verified the current bug and everything seems good.