Problem/Motivation
You can not add a state/province to address fields in events or profiles by default. In the US (and potentially other countries as well) this field is important.
Proposed resolution
"The field that needs to be added is administrative area. This corresponds to province, state, and region. My understanding is that address module is smart enough to add those fields conditionally the way it adds zipcode for United States and postal code for Australia."
Remaining tasks
- Export this default field configuration for events and profiles.
- Check if this does not create regressions for countries where this field is not needed.
- Create a pullrequest.
Comment | File | Size | Author |
---|---|---|---|
#6 | social-address-administrative-area-2835788-5.patch | 8.51 KB | robertragas |
#4 | social-address-administrative-area-2835788-4.patch | 8.53 KB | ribel |
available-address-fields.png | 46.03 KB | jaapjan |
Comments
Comment #2
ribelHi Guys!
I have created simple patch, which enables administrative area for Event and Profile address fields.
Also i have updated behat tests in order to check this new field.
Comment #3
ribelHere is updated patch.
Comment #4
ribelIn order to be consistent, I have also added administrative area for Group address field.
See patch and updated PR: https://github.com/open-social-eci/open_social/pull/11
Comment #5
nielsvandermolen CreditAttribution: nielsvandermolen at Open Social for Open Social commentedSeems to work well for a project we are working on and the code looks clean.
Comment #6
robertragas CreditAttribution: robertragas commentedAdded updated patch that can be used against the rc2
Comment #7
robertragas CreditAttribution: robertragas commentedComment #8
robertragas CreditAttribution: robertragas commentedComment #9
jochemvn CreditAttribution: jochemvn for Open Social commentedComment #10
jaapjan CreditAttribution: jaapjan commentedJust merged this in. Thank you!