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

Gender can be mapped from multiple columns #2437

Closed
richardolsson opened this issue Dec 20, 2024 · 1 comment · Fixed by #2439
Closed

Gender can be mapped from multiple columns #2437

richardolsson opened this issue Dec 20, 2024 · 1 comment · Fixed by #2439
Assignees
Labels
🐜 bug Something isn't working 🚪 entry-level Good for newcomers 🐟 Small Should be a quick-ish fix.

Comments

@richardolsson
Copy link
Member

Description

A new feature for mapping gender was recently added to the importer (PR #2401). The feature mimics how other features allow values to be mapped to tags, organizations or enum fields. But unlike tags or organizations, it should not be possible to import multiple genders to a single person, which it currently is.

Steps to reproduce

  1. Go to https://app.dev.zetkin.org/organize/1/people
  2. Click on Create
  3. Click on "Import people"
  4. Drag and drop the sample file below
  5. Map FNAME to First Name
  6. Map LNAME to Last Name
  7. Map GENDER to Gender
  8. Configure the gender mapping
  9. Click the checkbox next to PROFESSION
  10. Open the "Import as" dropdown

Sample file (CSV)

FNAME,LNAME,GENDER,PROFESSION
Clara,Zetkin,f,organizer
Richard,Olsson,m,programmer

Expected Behaviour

The dropdown should contain lots of options, some of which should be disabled because they have already been configured. The Gender field should be one of the disabled fields.

Actual Behaviour

The dropdown contains lots of options, some of which are disabled, but the gender field is not one of the disabled fields.

Screenshots (if you have any)

Configuring gender (Step 8)

image

The dropdown with Gender enabled (step 10)

image

@richardolsson richardolsson added 🐜 bug Something isn't working 🚪 entry-level Good for newcomers 🐟 Small Should be a quick-ish fix. labels Dec 20, 2024
@helihard helihard self-assigned this Dec 20, 2024
@helihard
Copy link
Collaborator

I might have come up with a solution, which seems to work on my end, but not sure it's a good one. Will PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐜 bug Something isn't working 🚪 entry-level Good for newcomers 🐟 Small Should be a quick-ish fix.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants