Opened 8 years ago
Closed 8 years ago
#2711 closed defect (bug) (fixed)
Make Sponsor province field optional
Reported by: | iandunn | Owned by: | coreymckrill |
---|---|---|---|
Milestone: | Priority: | low | |
Component: | WordCamp Site & Plugins | Keywords: | good-first-bug has-patch |
Cc: |
Description
In the Sponsors custom post type, the State / Province
field is required, but not all countries have intermediate regions like that. It should be optional for those countries.
On an implementation level, the simplest way is to just make the field always optional, but add a note asking users to fill it in if their country does have them.
See https://wordpress.slack.com/archives/C08M59V3P/p1491641449095148
Attachments (5)
Change History (9)
#2
@
8 years ago
- Keywords has-patch added; needs-patch removed
@iandunn Hows this for a first meta-ticket? :)
#3
@
8 years ago
In the https://meta.trac.wordpress.org/attachment/ticket/2711/2711.meta.3.diff I added a 'underwater' default state that is 'Not Applicable' to make sure it's always filled with something meaningful. It removed this string on the user-form to prevent them seeing the actual value.
Working on this right now. I have removed the required html attribute and added a description tag stating it's only required when your country uses states or provinces. I'll submit my patch in a moment.