Skip to content

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.

@KarinG KarinG merged commit d05a1b0 into colemanw:6.x Jun 3, 2025
4 checks passed
@KarinG
Copy link
Collaborator

KarinG commented Jun 3, 2025

Ok I've tested it - merging 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