-
Notifications
You must be signed in to change notification settings - Fork 2
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
Zoom webform issues #314
Comments
@mariacha In PR's: #319 https://github.com/wri/wriflagship/pull/1231 Note: in researching how to implement this I noticed the Pantheon docs mentioning |
#314: Add new IP Address element, update hook.
Ok, I tested this out on develop and it's ready to go to live. |
@cpmadera This change went up with today's deploy. Can you let me know if the IP addresses are coming through correctly when you get a moment? |
remote_addr
) does not match the value pulled by the[webform_submission:ip-address]
on the Pantheon live site only.Dev tasks
wri_zoom
module, add a new webform elementWebformElement
plugin that's "IP address". Examples can be found atweb/modules/contrib/webform/src/Plugin/WebformElement
.getValue
method to return$this->getWebform()->hasRemoteAddr()) ? $this->getRequest()->getClientIp() : ''
. That example was pulled from the code in webform,\Drupal\webform\WebformSubmissionForm::save
.[webform_submission:ip-address]
to use this plugin instead.The text was updated successfully, but these errors were encountered: