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

Family Migration needs tweaking #420

Open
bseeger opened this issue Sep 24, 2021 · 1 comment
Open

Family Migration needs tweaking #420

bseeger opened this issue Sep 24, 2021 · 1 comment
Labels
LAG post-launch wontfix This will not be worked on

Comments

@bseeger
Copy link

bseeger commented Sep 24, 2021

The family migration will not run successfully unless there is a value in the not-required Date field. If you leave it out of the spreadsheet you will get a vague error about the field being NULL when it should be a string.

It's most likely because the function called on the field, doesn't handle an empty field very well.

field_date:
    -
      plugin: explode
      source: date
      delimiter: '||'

The next field is field_authority_link, which should fail the same way except that strict is set to false. In the above, it defaults to true, which means the system makes sure that what it passes into explode is strictly a string. In the case where it's unset, it's a NULL and the system throws an error.

@mjanowiecki can work around this for now by including a date column with an empty string in the field. We should probably change the migration though. There's only one family in the pilot set, so it's not a huge need right now to change this.

@htpvu
Copy link

htpvu commented Nov 11, 2021

The fix would be a configuration fix in a yaml file. Check with @bseeger

@htpvu htpvu removed the follow-up label Nov 11, 2021
@htpvu htpvu added post-launch wontfix This will not be worked on labels Mar 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LAG post-launch wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants