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

Idea's for improving the csv logbook import process #2699

Closed
6 of 10 tasks
brendanheywood opened this issue Apr 11, 2017 · 4 comments
Closed
6 of 10 tasks

Idea's for improving the csv logbook import process #2699

brendanheywood opened this issue Apr 11, 2017 · 4 comments
Labels
5: Enhancement Build up on existing function that adds or improve functionality significantly Area: Logbooks Ticking & tick lists Feature: Import/export Logbook

Comments

@brendanheywood
Copy link
Member

brendanheywood commented Apr 11, 2017

I've just been playing with the csv import process and I think there is some low hanging fruit that would make this much more usable. The main issue I have had so far is the cognitive load trying to match up the columns. The process introduces the arbitrary meaningless labels like Col A, Col B, Col C but then you have to scroll up and down to see which columns these are actually talking about. It would be good to have the column selection directly linked with each column and not in a second set of fields.

Feedback on 'Upload CSV logbook' step

  • "Third Party Source" had some junk in it

Feedback on the mapping page:
This is what my import screen looks like when trying to re-import my own log book, I've zoomed right out:
image

  • it's very common for csv files to have lots of columns, the table layout should look and feel roughly like excel like this:
    image

  • to make it really easy to map between the columns and the values we should either

    • invert the selection so instead of selection which column to pull from, we have a dropdown above each column and select what value it represents, eg the dropdown has 'Country, Crag, Sector, Cliff ...'
    • we leave it as is, but we extend the label of each to be "Col A: Ascent label", "Col B: Ascent ID", "Col C: Ascent link" this assumes that the first row is the labels which is a pretty safe assumption, and even if it's not then you'll instead get real data which is just as useful for mapping. I'm guessing this one is easier to implement.
  • the 'Confirm configuration' drives me nuts, this serves no purpose at all here. If you want to confirm something then you need to be presented with a summary of what you've just done to validate it.

Feedback on Import dashboard:

  • when I got to this page I had absolutely no idea what to do next and the buttons made no sense to me. We need to rename some of these and add some help text to guide people through it.

Bulk assessment

  • not sure if things are working right on dev, I got the "Server currently process import files" message but I don't think it's processing, and clicking other buttons I got cids errors

Other issues with the export:

  • I think this is already known but the columns are screwed in the logbook export, the column don't match the data.
  • Also the urls used in the export don't have the proper url stubs
@brendanheywood brendanheywood added Area: Logbooks Ticking & tick lists 5: Enhancement Build up on existing function that adds or improve functionality significantly labels Apr 11, 2017
@nicHoch
Copy link
Contributor

nicHoch commented Apr 12, 2017

add more date formats

  • (d)d.(m)m.yyyy

is there some encoding auto detection we can utilize for special chars?

@Mdemaillard
Copy link
Collaborator

@nicHoch you improved multiple things recently, so have a good grasp on current status:
can you please check if some of the aboves are ticked ?

@nicHoch
Copy link
Contributor

nicHoch commented Jun 19, 2020

@Mdemaillard all done

can you please copy the issues for the export:

===========
Other issues with the export:

  • I think this is already known but the columns are screwed in the logbook export, the column don't match the data.
  • Also the urls used in the export don't have the proper url stubs

===========

to a better issue and close this afterwards

@nicHoch
Copy link
Contributor

nicHoch commented Jun 22, 2020

available with next release

@nicHoch nicHoch closed this as completed Jun 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5: Enhancement Build up on existing function that adds or improve functionality significantly Area: Logbooks Ticking & tick lists Feature: Import/export Logbook
Projects
None yet
Development

No branches or pull requests

3 participants