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

ERM-462 - CSV Language Set Upload and Export via JSON and XML #4

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

bayfaub
Copy link

@bayfaub bayfaub commented Oct 11, 2024

Changes:

  1. Added NeoTerm which is a stand-in model for Term and Context models that are being created in a different PR
  2. Added NeoTerms CSVupload form in Django admin
  3. Added NeoTerms export as file via JSON or XML

How to Test:

  1. Start project and sign into admin and verify that NeoTerm form exists with three buttons: Upload CSV, Export Terms(XML), Export Terms(JSON)
  2. Click on Upload CSV and Upload a CSV File. Test with the 5 five files that you can get from @bayfaub
  3. To verify that terms have actually been created navigate to http://localhost:7474/browser/ and use neo4j as username and password as password. Then run match(n) return n to view any nodes that have been created. you can run this query every time you upload a valid or invalid CSV to validate node creation or the lack thereof.
  4. You can also export the nodes as json or xml by clicking either of the Export Terms buttons.

@bayfaub bayfaub requested review from FlorianTolk, vbhayden, hsmith-adl and EdblackGitHub and removed request for EdblackGitHub October 11, 2024 05:53
bayfaub and others added 8 commits October 11, 2024 13:03
Prevent non-csv files to be uploaded
Dynamically redirect on upload
* Implement exceptions
* Dynamic required columns
* Split into smaller funcs
@bayfaub bayfaub marked this pull request as ready for review October 16, 2024 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants