update on what's happening in master

[Tomaz Canabrava]

Em 07/01/2015 11:20, "Miika Turkia" <miika.turkia at gmail.com> escreveu:
>
> > On 07 Jan 2015, at 12:50, Dirk Hohndel <dirk at hohndel.org> wrote:
> >
> > Since FB is taking their time with getting back to us... I decided to
ask
> > Tomaz to work on a really cool UI for the CSV import dialog. I just
pushed
> > his massive amount of work to master and would like people to play with
it
> > (daily builds will be available fairly soon).
>
> Excellent. Soon there will be no traces of my "GUI work" left. And
everybody wins :D
>
> > To use this you need a .csv file to import. You can easily create this
by
> > exporting a few of your dives to CSV.
> >
> > Then go to Import and select that file. You'll see the new UI that
allows
> > you to look at the data you are about to import (make sure you have
picked
> > the correct separator or this won't work). And then you can drag and
drop
> > the field names in the upper area to the top row in the spread sheet in
> > the lower area. I find this super intuitive and really cool. Kudos to
> > Tomaz for this initial implementation.
> >
> > Of course there are quite a few caveats right now:
> >
> > a) You'll notice that there are some discrepancies between the fields we
> >   export and those we can import. I added divemaster which I immediately
> >   noticed but then realized that there are more differences (suite,
> >   rating, viz, tank data). So this part needs more work.
>
> Preset Subsurface import has all these fields but there was no way to add
them all into the old configurable UI. I might not have time during the
next 2 weeks to improve the configurable import. Diving comes first.
>
> > b) The preset types that we had in the old UI are still offered, but
they
> >   no longer populate the colums correctly. This will need to be added.
>
> UI should be mostly ignored by these. At most, the field separator, unit
and day format? are respected. Thus my first guess is that the xslt call is
messed. Make sure you have correct file name. I'll try to take a peak into
the new ui once not on the road (air).

Ui must know these, and I didnt do that part yet, what needs to be done:
when changing the presset, we must remove the column names from one table
tô the other ( or reorder )

Easy to do, I just didn't had time to do it yet.
>
> > c) I can't get it to actually do the right thing, i.e. it doesn't import
> >   things for me. I believe this worked for Tomaz so it's possible I
broke
> >   something when trying to do minor cleanups in the code. If someone has
> >   the time to take a look at where this fails, that would be very nice
> > d) I occasionally can get this to crash. I fixed a couple of situations
> >   where this was easy to reproduce but I think there are more. If you
can
> >   find a reproduceable way to crash this (or make it behave incorrectly
> >   in other ways), please post here so we can fix this.
> >
> > I'll admit that I'm really excited about this UI - I think this a very
> > cool way to allow people to import from CSV - another highlight for 4.4
> >
> > As indicated above, FB is slow to respond, my next trip is coming up, so
> > 4.4 is delayed until closer to the end of the month...
>
> Then I should be able to do some real testing on DC import befor the
release :)
>
> miika
> _______________________________________________
> subsurface mailing list
> subsurface at subsurface-divelog.org
> http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20150107/22b7260e/attachment.html>