[mozillians] Province/State should not be required for some regions

VERIFIED FIXED in 2013-06-27

Status

Participation Infrastructure
Phonebook
VERIFIED FIXED
6 years ago
5 years ago

People

(Reporter: tomer, Assigned: John Kim [snoozes])

Tracking

(Blocks: 1 bug)

other
2013-06-27

Details

(Whiteboard: [kb=1004654] [good first bug][lang=python])

(Reporter)

Description

6 years ago
In some regions in the world it doesn't make sense to specify Province/State, and people in such locations never fill the Province/State fields. 

In mozillians.org this field is required, and I find it useless to set it to a dummy string. Instead, I'd suggest having an option to keep it blank or make it required only to specific countries (such as the US, for example).
Yes, this feels like one more case of US-centrism, and I think we can avoid uselessly offending people by not requiring the state :).

Comment 2

5 years ago
I was going to file this bug too, but noticed that it's already here. Please fix this annoyance. That field should indeed be optional.
Currently 'Province/State' is required only if you specify a city. If you don't specify a city you don't need to fill in 'Province/State' Do you think that we should make field optional in all occasions?

btw I believe 'Region' is more appropriate to 'Province/State'

Comment 4

5 years ago
(In reply to Giorgos Logiotatidis [:giorgos] from comment #3)
> Currently 'Province/State' is required only if you specify a city. If you
> don't specify a city you don't need to fill in 'Province/State' Do you think
> that we should make field optional in all occasions?

Yes, please. In many countries regions aren't signified as much as they are in US. E.g. in Lithuania, the region is usually skipped unless you live in a small town or a village, in which case the region might be needed as a clarification.

> btw I believe 'Region' is more appropriate to 'Province/State'

http://youtu.be/ussCHoQttyQ ;)

Updated

5 years ago
Blocks: 791954
Whiteboard: [good first bug]
Whiteboard: [good first bug] → [good first bug][lang=python]
Assignee: nobody → john_hs_kim
Status: NEW → ASSIGNED
(Assignee)

Comment 5

5 years ago
Hello.

I submitted a pull request on the following change:
  Deleted the requirement for the Province/State in apps/phonebook/forms.py
(Assignee)

Comment 6

5 years ago
I resubmitted the pull request with the message change.  Sorry for the inconvenience.
(Assignee)

Comment 7

5 years ago
I submitted a new commit where we added a new test case in phonebook/test/test_edit_profile.py and made a small change in phonebook/forms.py.
Hopefully forms.py works properly this time around.
(Assignee)

Comment 8

5 years ago
Hello.
I attempted at squashing all my commits but I believe that I did it wrong.  Could someone help me on how to fix this or how to squash properly?
(Assignee)

Comment 9

5 years ago
Hello.
I submitted a new commit.  Really hoping that the squash was done correctly.
Whiteboard: [good first bug][lang=python] → [kb=1004654] [good first bug][lang=python]
Fixed:

https://github.com/mozilla/mozillians/commit/9927ece0240208a49878d9799fe87a0cfc573f3e


Thanks snoozes!
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2013-06-27
Can you provide minimal steps to test fro QA. I'm not sure I understand the scope of the change.
(In reply to Matt Brandt [:mbrandt] from comment #11)
> Can you provide minimal steps to test fro QA. I'm not sure I understand the
> scope of the change.

It looks like the fix from Giorgos makes the province/state field optional instead of required. The only test for now is that you can create/edit a profile without providing a value in the province/state field.

It would be nice to require the province/state for users who are in the United States. Are there other regions where we should require a province/state?
(In reply to William Reynolds [:williamr] from comment #12) 
> It looks like the fix from Giorgos makes the province/state field optional
> instead of required. The only test for now is that you can create/edit a
> profile without providing a value in the province/state field.

Thanks William, that's what it appeared to be but I didn't want to assume.

> It would be nice to require the province/state for users who are in the
> United States. Are there other regions where we should require a
> province/state?

Honestly .. I don't know :-) perhaps we can reach out to our localizers for some tips.
QA verified on stage - only the Country field is required during new user registration and when editing an existing profile.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.