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.

Sign up for GitHub

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

Jump to bottom

Bugfix: Format data values for multi-choice fields #48

Merged
merged 3 commits into from
Mar 7, 2023
Merged

Bugfix: Format data values for multi-choice fields #48

merged 3 commits into from
Mar 7, 2023

Commits on Mar 7, 2023

  1. Bugfix: Format data values for multi-choice fields 

    This addresses https://phabricator.wikimedia.org/T331262

As I wasn't having any luck creating a custom schema that would
correctly handle both strings and lists, I opted to accept a string
and convert it into a list.

Unfortunately, Flask did not do a good job of converting the resulting
data to a valid JSON, so I had to do it manually.
    @nbarnabee
    nbarnabee committed Mar 7, 2023
    Configuration menu
    Copy the full SHA
    e294691 View commit details
    Browse the repository at this point in the history

  2. Commit variable name fix and CI tweak

    @nbarnabee
    nbarnabee committed Mar 7, 2023
    Configuration menu
    Copy the full SHA
    f6eb95b View commit details
    Browse the repository at this point in the history

  3. Change function name and update import 

    This change is completely unrelated to this PR.
    @nbarnabee
    nbarnabee committed Mar 7, 2023
    Configuration menu
    Copy the full SHA
    6bfa04e View commit details
    Browse the repository at this point in the history