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

Fix multiple checkbox from relation ship custom field not loading #1030

Open
wants to merge 9 commits into
base: 6.x
Choose a base branch
from

Conversation

peetCreative
Copy link
Contributor

For detailed information please read:
https://www.drupal.org/project/webform_civicrm/issues/3494307

@KarinG proposed to open this PR.

Open TODO: Write tests.

@peetCreative
Copy link
Contributor Author

@KarinG I implemented test, which passed partially.
For the first two, I have the impression, that it's not failing because of my new tests or code, what do you think?

@KarinG
Copy link
Collaborator

KarinG commented Jan 28, 2025

Well done - thank you for persevering. I will add this to my list to review.

@KarinG
Copy link
Collaborator

KarinG commented Feb 2, 2025

Hey Peter - looking at this now. Awesome that you got the test in!

My notes ->

1. Reproduced:
Checkboxes on Contact entity fields retrieve the correct values from the database -> Favourite Colours: red and white.
Checkboxes on Relationship entity fields are not retrieved -> Deutschland, Canada

Note: Partner 1 : Existing Contact -> Current User and Partner 2: Existing Contact -> Default value -> Relationship to: Partner 1 -> Specify Relationship

image

2. Reproduce in a test.
Making some small edits to your test - will push them to your branch.

@KarinG
Copy link
Collaborator

KarinG commented Feb 2, 2025

Ok waiting for these tests to fail. Back in a bit.

@KarinG
Copy link
Collaborator

KarinG commented Feb 2, 2025

civicrm extention dir is down -> "error_message": "It is not possible to contact the CiviCRM extensions directory. You may be missing out on the latest updates to extensions. Check that you can view the <a href="https://civicrm.org/extdir/ver=5.82.beta1|cms=Drupal8|ready=\">extension feed. If that works check that your webserver can make external HTTP requests."

@KarinG
Copy link
Collaborator

KarinG commented Feb 2, 2025

image

@KarinG
Copy link
Collaborator

KarinG commented Feb 2, 2025

Ok test now fails exactly where we need it to fail ->

image

Now adding your code fix back in:

@KarinG
Copy link
Collaborator

KarinG commented Feb 3, 2025

All tests are passing 🥳

And this is the test browser output ->
/screenshots.10.3._5.81._8.2%20(1)/Drupal_Tests_webform_civicrm_FunctionalJavascript_ContactRelationshipTest-8-87009787.html

image

I will pull this into webform-civicrm.io this week to run some final live checks and then we can merge it.

@peetCreative
Copy link
Contributor Author

Is there some progress?

@KarinG
Copy link
Collaborator

KarinG commented Feb 24, 2025

I've not had a chance to live test/QA yet. I will get back to it (as I already spent time on this PR/editing your branch), but If you want to fast track this and if you're able to sponsor 1-2h of time then I can ask one of my team to help QA/review it and then I can release a new 6.x version.

@peetCreative
Copy link
Contributor Author

Thanks for the notice. I fear as I'm employed myself for this project I cannot. :|
However it is not urgend by now.

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.

2 participants