Opened 7 years ago

Closed 7 years ago

#1555 closed bug (fixed)

Contact info update usability issues

Reported by: planetcruiser Owned by: planetcruiser
Priority: blocker Milestone: 0.5.5 - bugfixing
Component: BW Profile Keywords:
Cc:

Description (last modified by planetcruiser)

Issue:

  • If no zip code is filled in, the profile information won't be saved and users get confused (see forums)

There are the following additional issues with this:

  • Mandatory fields in profile are not marked as such
  • Error message is not specific enough, should mention "zip"
  • Contact Information tab is not opened on error
  • Error message appears in the wrong place (next to Street)
  • The word "zip" for postal code is not recognised everywhere

Reproduce:

  1. Edit profile
  2. Leave zip field empty
  3. Save profile
  4. Error appears

Solution:

  • Mark mandatory fields
  • Open correct tab on error
  • Make error message more specific
  • Fix error message position
  • Rename "zip" to "postal code", which is more descriptive

Related tickets:

Change History (11)

comment:1 Changed 7 years ago by planetcruiser

on making the postal code optional: some countries don't have postal codes, like panama and ireland, or they are not used as much, like in turkey .. i know this is hard to imagine coming from central europe, but that's how it is. :)

we could consider keeping a list of countries that don't have official postal codes and allow it to be empty there. but then i just saw that we don't seem to store the country of a user in a separate field - argh

comment:2 Changed 7 years ago by planetcruiser

  • Description modified (diff)
  • Summary changed from Profile won't update without zip code to Contact info update usability issues

i decided to create a separate ticket for making the postal code optional: #1556

comment:3 Changed 7 years ago by planetcruiser

  • Status changed from new to accepted

i will look at solving this

comment:5 Changed 7 years ago by planetcruiser

  • Resolution set to fixed
  • Status changed from accepted to closed

i added english translations for "SignupErrorInvalid*" strings and "Save Profile"

comment:6 Changed 7 years ago by planetcruiser

also added translations for german

comment:7 Changed 7 years ago by globetrotter_tt

  • follow_up changed from none to test on alpha

Shouldn't we set the "follow up needed" field to "test on alpha", so that people know which tickets need attention?

comment:8 Changed 7 years ago by globetrotter_tt

  • Component changed from unknown to BW Profile

comment:9 Changed 7 years ago by planetcruiser

  • Resolution fixed deleted
  • Status changed from closed to reopened

good point

comment:10 Changed 7 years ago by planetcruiser

actually, until we have a better testing workflow i think it's ok to close tickets and ask testers to look at closed tickets for this milestone, see http://trac.bewelcome.org/milestone/0.5.5%20-%20bugfixing ("closed:")

comment:11 Changed 7 years ago by planetcruiser

  • Resolution set to fixed
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.