-
Notifications
You must be signed in to change notification settings - Fork 14
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
GeoBrowser(CSV) does not display a CSV file #96
Comments
Ok, so the documentation says these fields are mandatory:
Do any other columns matter, or are they silently ignored? Would any csv work as long as it has these columns? |
I do not know either :-) Maybe you would need a title row? --> also improve error handling, and maybe documentation, too! |
It does work with the following file (created with the Datasheet Editor): https://cdstar.de.dariah.eu/dariah/EAEA0-0CB7-1F90-DF1B-0 Without time fields you just get places visualized. |
See a repository file here: https://trep.de.dariah.eu/1.0/dhcrud/21.T11991/0000-001B-4223-1/index We do call https://switchboard.clarin-dev.eu/switchboard-test/#/trep/https%3A%2F%2Ftrep.de.dariah.eu%2F1.0%2Fdhcrud%2Fhdl:21.T11991%2F0000-001B-4223-1%2Fdata/application%2Fvnd.dariahde.geobrowser.csv from this index page, and we do not get the correct mimetype from swotchboard? Something wrong with the URL? |
But we give the correct mimetype with the switchboard call, and this mimetype is existing in the Switchboard, too. Don't you take the minetype from the API call, if existing? |
In some cases the provided mimetype cannot be trusted and because of that we currently ignore it. I think you're right that we should not ignore it in this particular case (clarin-eric/switchboard#197) |
@fungunga We tested the GeoBrowser with a csv file but it does not display anything on the map, only the first two columns of the csv file (attached, renamed to .txt because github would not attach it otherwise).
all-test-geo.csv.txt
The text was updated successfully, but these errors were encountered: