search

jqnatividad / ckanext-officedocs

MS Office and OpenOffice Resource viewer
GNU Affero General Public License v3.0
15 stars 23 forks source link

Add support for TSVs #7

Closed jqnatividad closed 5 years ago

jqnatividad commented 9 years ago

Officeviewer.com doesn't support TSVs, though Excel can open them directly.

A way around this is to use the Zoho Sheet Viewer which has the added benefit of also support OpenOffice spreadsheets.

cc @rossjones @talos

rossjones commented 9 years ago

Only seems to do a full-screen view, triggered by https://sheet.zoho.com/sheet/view.do?url=.... and then redirecting to a full-browser editor. Which is neat I guess. Perhaps there should be an 'Open with...' option to open in your choice of online services?

Does the default data grid view in CKAN not support TSV? That seems like the obvious place to do the fix.

talos commented 9 years ago

Since TSV is just a CSV with a tab instead of comma as delimiter, any CSV importer that can take a custom argument for delimiter should be able to handle them... Is it just a matter of renaming the TSV to CSV, and letting the importer figure out the proper delimiter?

jqnatividad commented 8 years ago

Agreed. TSV is just a CSV variant.

The main reason I opened this issue is so that we can view a TSV that's actually just a URL that CKAN is pointing to, so we don't need to re-upload it to the CKAN.

The full context of this issue is here - http://data.beta.nyc/dataset/nyc-2011-housing-and-vacancy-survey

jqnatividad commented 8 years ago

TSVs now supported by the datapusher with CKAN 2.5. :)

http://docs.ckan.org/en/latest/changelog.html

Still not closing this issue as admin may still prefer to configure a resource view of TSVs using this plugin.

jqnatividad commented 5 years ago

Closing this issue as TSVs are supported.