Eliminate external JSON export script - Githubissues

hackforla / peopledepot

A project to setup a datastore for people and projects at HackforLA. The link below takes you to the code documentation

https://hackforla.github.io/peopledepot/

GNU General Public License v2.0

5 stars 24 forks source link

Eliminate external JSON export script #269

Closed anandramakris closed 3 months ago

anandramakris commented 3 months ago

Fixes #144

What changes did you make?

Changed convert script so it gets data from a csv spreadsheet directly. The data is converted to JSON inside the file, removing the step of having to run the Google Apps script for JSON data.
Made sure id and uuid values are cast to int rather than string while converting from JSON to Python.
Modified documentation to show how to export the spreadsheet

Why did you make the changes (we will use this info to test)?

The external JSON script is unnecessary.

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

Visuals before changes are applied

![image](Paste_Your_Image_Link_Here_After_Attaching_Files)

Visuals after changes are applied

![image](Paste_Your_Image_Link_Here_After_Attaching_Files)

fyliu commented 3 months ago

@anandramakris I think you misunderstood the goal of the issue. The intention was to eliminate the extra step of exporting to a file and then reading that file back. Python can read the data from the google spreadsheet without the file export step.