You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Since we include Middle Name in out profiles, it would be very usefull for us to include the middle name field in this extension. I still need to figure out submitting PR's etcetera. Be gracefull on me. This is the code change I propose. Thanks a lot for your hard work. :-) Regards, RIchard
Hi,
We don't use middle name and that's why I didn't think about including it, but this would be a welcome addition!
PR isn't that scary, you do a fork, do the change on your fork, push it and voila, it's ready for a PR ;)
github has a decent documentation on the topic.
I'm going to be moving between cities in the next two weeks but will check my email regularly.
The disabling of the middle_name field (which 50% of dutch names use, 50% not) was not working when the value was empty. Found the line of code that caused it and set up this PR: #19
Since we include Middle Name in out profiles, it would be very usefull for us to include the middle name field in this extension. I still need to figure out submitting PR's etcetera. Be gracefull on me. This is the code change I propose. Thanks a lot for your hard work. :-) Regards, RIchard
noverwrite.js
noverwrite.php
+foreach (array( 'first_name', 'middle_name','last_name') as $f) {
The text was updated successfully, but these errors were encountered: