akvo / akvo-lumen

Make sense of your data
https://akvo.org/akvo-lumen
GNU Affero General Public License v3.0
63 stars 18 forks source link

Add walkthrough to new dataset UI #2749

Closed janagombitova closed 4 years ago

janagombitova commented 4 years ago

Context

Why do we have this issue? What are we trying to solve? 

We want our products to make it easy to work with data. This also means that it should be easy to work with the products and thus easy to get started with different product features. Having a way to explain what different buttons do and how they behave is a nice way to get started with the tool.

How will this benefit the users?

How will this benefit Akvo? 

Opportunity 

Current status quo

Currently, the only way users learn about the product pieces (or changes to them) is by exploring and reading the support documentation. It is ok but there are more friendlier ways to introduce features and changes to users.

A while back we decided to use https://introjs.com/example/hello-world/index.html in Flow to walk users through new features.

The idea 

The idea is to do the same for Lumen. To use IntroJs and to walk the users through the dataset view. Why? With supporting RQGs we are changing the data grid quite a lot (view, navigation, how derived columns behave). So this is a good opportunity to explain the changes to existing users and explain the data grid to new users.

Screen Shot 2020-07-06 at 11 06 23

Data set view components

Next steps 

Kiarii commented 4 years ago

I think with this for now we should prioritize one main item

janagombitova commented 4 years ago

All great points @Kiarii, well taken. My main goal with this issue to communicate the new data groups change somehow in the UI. So let’s indeed prioritize this. What would your suggestion be?

Could you and @marvinkome define and inplement the tool tips based on the design? Could we add one for the repeated group icon too?

Kiarii commented 4 years ago

Tentative idea: https://www.figma.com/proto/XQtjy781ugZ8dmmaA9hWnO/Lumen-style-guide?node-id=124%3A0&viewport=4413%2C1012%2C1&scaling=min-zoom

janagombitova commented 4 years ago

The idea is great! To the point and all that we need.

So we have two tasks:

  1. Implement the tooltips.
    • Can we rethink the description of Transformations? The one there now is Transform dataset. This not really what happens. The user creates/derives new columns based on the data. Any other ideas?
    • Example: Akvo Flow datasets are split into data groups. A data group is based on form question groups and submission metadata.
  2. Implement the 2 step walkthrough
    • For the 1st step I would suggest to revise the text as not all datasets are split into data groups, only those coming from Flow. So we need to make this clear. And the metadata group is not for the form, but about each submission.
    • For the 2nd step I suggest to write is as "instruction" rather than as an explanation: Select a data group to see the data in the grid. or is this not good practice?
    • How about adding another sentence here saying: Derived columns will appear at the end of the data group. or something in that line to explain the change in the behaviour of the derived columns?
Kiarii commented 4 years ago

On item 1

Item 2

Kiarii commented 4 years ago

as a side note, I still think user created cols shd be noted as derived cols rather than transformations; as we have seen transformations refer to every change in structure or appearance applied to the dataset even when such do not result to a new col.. Put another way, a dataset can consist of 100+ transformations without there being a single user created col

janagombitova commented 4 years ago

@Kiarii all valid notes. I think for the initial iteration, the derived columns will remain grouped in the Transformation group and we will only move the RQG derived columns into the group. The next step (once supporting RQGs is handled) will be to figure out what to do with the derived columns.

So for now, for this issue:

Kiarii commented 4 years ago

From one of the test sessions, we realized that it would be valuable to mention that in the light of presenting a dataset by data groups, all data relations are maintained, to assure Salim and co. that these are not lost or broken.

Basically we would need to mention this in the first screen when we introduce the new layout, I am still trying to think how we might phrase this - suggestions are welcome..

janagombitova commented 4 years ago

Looking good @marvinkome 👍 well done.

Can you specify here what backend changes are needed to finish up this task?

janagombitova commented 4 years ago

Works as expected