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

dev/core#3160 fix inability to import 'just contactid' and add to group #23756

Merged
merged 1 commit into from
Jun 10, 2022

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Jun 10, 2022

Overview

dev/core#3160 fix inability to import 'just contactid' and add to group

Before

  • pre 5.51 - trying to import just a contact ID and add to group fails due to missing mandatory fields
  • 5.51 rc without this - the add to group stopped happening very recently stopped working
  • no test

After

Importing 'just an id' works, tested

Technical Details

The add to group happens per batch in the post_actions

Comments

https://lab.civicrm.org/dev/core/-/issues/3160

@civibot
Copy link

civibot bot commented Jun 10, 2022

(Standard links)

@civibot civibot bot added the 5.51 label Jun 10, 2022
@totten
Copy link
Member

totten commented Jun 10, 2022

pre 5.51 - trying to import just a contact ID and add to group fails due to missing mandatory fields

At some point while testing import_to_group, I got that same error:

Screen Shot 2022-06-10 at 3 19 17 PM

I thought that happened while running import_to_group, but it could be that I needed a more up-to-date base. I merged locally with head, reset DB -- then it ran cleanly.

@totten totten merged commit d76cb82 into civicrm:5.51 Jun 10, 2022
@eileenmcnaughton eileenmcnaughton deleted the import_to_group branch June 10, 2022 22:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants