kolibri icon indicating copy to clipboard operation
kolibri copied to clipboard

CSV import supports email address as identifier but Users list UI doesn't render it properly

Open intelliant01 opened this issue 4 years ago • 8 comments

Observed behavior

CSV import instructions at Facility - DATA - Import and export users - View spreadsheet format reference suggests using email address as an Identifier -

image

But an email address overlaps with gender field even for short emails in the Facility - USERS list -

image

Expected behavior

Identifier field's value should not overlap with gender field value.

User-facing consequences

Unreadable data

Errors and logs

Steps to reproduce

Context

Kolibri 0.14.0

intelliant01 avatar Aug 10 '20 15:08 intelliant01

@intelliant01 could you elaborate a bit more where is the issue? I don't quite understand it. If you could attach a csv to test it, it would be great.

jredrejo avatar Aug 10 '20 16:08 jredrejo

@jredrejo This isn't an issue with CSV import, it's just that the Identifier column on the 'Facility > User' field has a max-width of 120px, which isn't wide enough for long email addresses.

https://github.com/learningequality/kolibri/blob/0beb63c921f4568a228c03fb96f444319818016d/kolibri/plugins/facility/assets/src/views/UserTable.vue#L265-L268

jonboiser avatar Aug 10 '20 17:08 jonboiser

@jredrejo the issue is not per se with the CSV import process. Just edit a user's profile and set an average length email address and choose a gender for any user and we have the email address and gender values overlapping.

I am referring to the CSV import because it is only in those instructions that it suggests using an email id for the identifier field.

intelliant01 avatar Aug 10 '20 17:08 intelliant01

@radinamatic this one seems to be resolved in 0.15.6 as a slider is displayed if the value is too long:

2022-08-03_15-20-28

pcenov avatar Aug 03 '22 12:08 pcenov

Closing as fixed.

rtibbles avatar Aug 03 '22 15:08 rtibbles

@rtibbles actually we were just discussing with Radina that this one is still not fixed in Firefox and IE so it should be reopened:

IE11: 2022-08-03_18-15-39 Firefox: 2022-08-03_18-19-16

pcenov avatar Aug 03 '22 15:08 pcenov

Still a few bumps remain to flatten...

radinamatic avatar Aug 03 '22 15:08 radinamatic

Ahah.

rtibbles avatar Aug 03 '22 15:08 rtibbles