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

Warning about missing choice label when choice has an image label #679

Closed
lognaturel opened this issue Jan 9, 2024 · 2 comments
Closed
Milestone

Comments

@lognaturel
Copy link
Contributor

Software and hardware versions

pyxform v2.0.0

Problem description

Warning about missing choice label when choice has an image label

Steps to reproduce the problem

  1. Convert a form with a choice list that only has image labels (no text labels) such as this one
  2. Notice the warnings:
On the choices sheet there is a option with no label. [list_name : images-only] 
On the choices sheet there is a option with no label. [list_name : images-only] 

Expected behavior

If choices in the list have images, there should be no warning. This may not be compatible with some of the new approach to warnings that we've taken so they may need to be reconsidered.

Other information

Things you tried, stack traces, related issues, suggestions on how to fix it...

@lognaturel lognaturel added this to the Next milestone Jan 9, 2024
@lindsay-stevens
Copy link
Contributor

lindsay-stevens commented Jan 10, 2024

If choices in the list have images, there should be no warning.

@lognaturel The warning that this comes from, which is a check for a choice name and label value, has been in the code base since 2012. On the XLSForm.org and docs.getodk.org, the label column is described as being one of 3 mandatory choices columns. Are we saying now that the label is (conditionally?) optional? Are any media OK in place of a label - or just images?

This may not be compatible with some of the new approach to warnings that we've taken so they may need to be reconsidered.

Sorry but what is this referring to?

@lognaturel
Copy link
Contributor Author

Confirmed that this is old behavior. I thought it was related to changes to language warnings but it's not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants