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

Update Import.php #45

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

petednz
Copy link

@petednz petednz commented Jan 11, 2022

This was a fix i see J. did that fixed a problem for a client. I just had to do some digging as they were again having an issue, but this time was their data. so having found this. and while being a bit clueless on what it fixed, i thought I should put it here for others to review.

This was a fix i see J. did that fixed a problem for a client. I just had to do some digging as they were again having an issue, but this time was their data. so having found this. and while being a bit clueless on what it fixed, i thought I should put it here for others to review.
@petednz
Copy link
Author

petednz commented Jan 17, 2022

The client had said

error "Error with entity "create"! (Mandatory key(s) missing from params array: one of (case_type, case_type_id)). the CSV file contains those fields, I've tried using the case type ID, case type label, case type name and I still get this error

and all I know is after this patch was created/applied, the import worked for them. Luke said

it looks like the code updates the validate function to allow for data to have api.*.get calls and allow them. I assume this means you can do an api lookup as part of the import

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

Successfully merging this pull request may close these issues.

1 participant