IntersectMBO / govtool

🔩 GovTool and utilities monorepo.
https://gov.tools
Apache License 2.0
13 stars 7 forks source link

Username hidden from user #1381

Open m-i-k-e opened 3 months ago

m-i-k-e commented 3 months ago

Context & versions

staging sanchonet

Steps to reproduce

  1. Go to homepage of Govtool or anywhere in Govtool
  2. Look for my username on screen
  3. Go to PDF homepage
  4. Look for my username on screen

Loom: https://www.loom.com/share/5dbc10ac4bef48ada2b9ad91dd6e5568?sid=2fcd11f3-9728-4035-aec9-ee90fbd2a096

Actual behavior

Cannot determine my username

Expected behavior

GovTool clearly shows me 'who I am' in relation to my interactions with it.

bosko-m commented 2 months ago

This should be a wider discussion that needs to happen on how we are going to handle and display username within the whole GovTool app.

m-i-k-e commented 2 months ago

@bosko-m My view on this: We need a mini-ux project to merge the PDF and GovTool. The designer did not properly think through the fact that these are the same application (PDF is merely functionality in GovTool). This is why we are coming up against several 'confusing' errors when these are merged into one.

Without analysing this 'merge', we will not be able to really fix this nicely

In general UX terms, if a user has a username, then they need to be able to see it/know it. I have no idea why it's not shown to the user. Users need to know who they are.

Ryun1 commented 2 weeks ago

Yeah we should fix this.... @teske00 do you mind having a look

teske00 commented 2 weeks ago

@Ryun1 Currently, it is structured that the username exists only at the PDF level because the username is created and stored on the PDF backend. It is possible to display the username somewhere on the PDF frontend, we just need a decision from maybe @m-i-k-e , where to put it

m-i-k-e commented 2 weeks ago

OK, @Ryun1, This is another design one. I'll put it in for doing.

l-br1 commented 2 weeks ago

In theory the plan was to add the user name above the wallet on the left side nav as soon as we implement the new wrapper