Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Downloadable organization data #596

Open
2 of 3 tasks
sseerrggii opened this issue Mar 9, 2021 · 9 comments
Open
2 of 3 tasks

Downloadable organization data #596

sseerrggii opened this issue Mar 9, 2021 · 9 comments

Comments

@sseerrggii
Copy link
Contributor

sseerrggii commented Mar 9, 2021

Some organizations want to analyze their data, in past we proposed to extend reports #259

Also in #594 we add some fields to these reports

These reports are commonly used only for printing in order to facilitate participation in TimeBanks for people that have difficulties using the TimeOverflow application (old people, for example)

So maybe in the future we could keep this reports simple (printable version) and add another option for admins like "download organization Data" .zip with all CSV of data of your Organization: Users/Posts/Categories/Tags/Transactions... with all fields in use avoiding those that are only used internally in the app and do not provide data for analysis (such as password, tokens, ...)

PLAN

  • Add the missing reports, like Transfers (👉🏼 Add Transfers Report  #599)
  • Add more and missing data in CSVs
  • Add a new option Download all under the hood we only need to call each CSV Report class and create a ZIP with all generated files (👉🏼 Reports: Download all #620)
@sseerrggii sseerrggii changed the title Downloadable organizarion data Downloadable organization data Mar 9, 2021
@markets
Copy link
Collaborator

markets commented Mar 12, 2021

That would be very useful, I agree @sseerrggii. Maybe we can just do it this way (to reuse more parts):

  • Add the missing reports, like Transfers
  • Add missing attributes in CSVs
  • Add a new option Download all, under the hood we only need to call each CSV Report class and create a ZIP with all generated files (using this gem https://github.com/rubyzip/rubyzip)

Something like this:

Captura de pantalla 2021-03-13 a las 1 24 43

@sseerrggii
Copy link
Contributor Author

I think @franpb14 wants to take this :)

@franpb14
Copy link
Collaborator

Yes, I would like to take this issue.
Does the current status of the issue coincide with the pull request #599?

@markets
Copy link
Collaborator

markets commented Apr 10, 2021

Hi @franpb14 👋🏼

In #599 I added the missing Transfer report and in #594 I added some missing relevant attributes. So, to finalize a 1st version of this project we need to implement the latest point in #596 (comment), basically add the Download all option.

franpb14 added a commit that referenced this issue Apr 16, 2021
The new update allows the administrator to download a zip containing all the current CSV documents. Issue #596.
@franpb14
Copy link
Collaborator

Hi, in this commit you can find the feature working, but I have a doubt regarding the require 'zip': I think it shouldn't appear in the file 'reports_controller.rb' but I don't know where else to put it.

@markets
Copy link
Collaborator

markets commented Apr 17, 2021

hi @franpb14 👋🏼 since it's the place from where it is invoked, is fine to have it there.

franpb14 added a commit that referenced this issue Apr 20, 2021
All the new methods that I implemented are now covered with the exception of the rescue block. Issue #596
@franpb14
Copy link
Collaborator

franpb14 commented Apr 20, 2021

I didn't close this Issue with a magic comment because I discussed with @sseerrggii the possibility of an additional phase (more data in csvs from the Zip). This phase would start when I implement another functionality that I have defined that would bring more value.

@markets
Copy link
Collaborator

markets commented Apr 20, 2021

Ok @franpb14! this is probably the 2 bullet point in the checklist here #596 (comment), right?

@franpb14
Copy link
Collaborator

Yes, I think so @markets

franpb14 added a commit that referenced this issue May 22, 2021
…ding

A single CSV to allow easy linking. 3 attributes have also been removed from the other reports. Issue #596.
franpb14 added a commit that referenced this issue May 22, 2021
I've also adapted the tests of members and offers decorators. Issue #596.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants