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
feat: import lists from excel (DSP-1341) #48
Conversation
# Conflicts: # knora/dsplib/models/connection.py
# Conflicts: # knora/bitstreams/test.csv # knora/dsp_tools.py # knora/test-onto.json
@lrosenth please add |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See my review comments.
Also note the following things I could not comment directly into the files:
- please add
.DS_Store
to.gitignore
- please ensure not to include any sensitive data in CSVs
- be sure not to include any copyright protected files as test data (I haven't checked images and audio, but PDF is an NZZ article which is not public domain)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- in general
BUILD
files should be namedBUILD.bazel
- could you please clean-up a bit the
/knora
folder and move the test data to another directory. It has become very messy. Only data that needs to be shipped, should be part of the/knora
folder.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
Added support for hierarchical lists in excel files and updated the documentation to make it more consistent and a bit easier to navigate.