Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

street address? #76

Closed
danvideo opened this issue Jul 9, 2014 · 25 comments
Closed

street address? #76

danvideo opened this issue Jul 9, 2014 · 25 comments

Comments

@danvideo
Copy link

danvideo commented Jul 9, 2014

Nice work. I was wondering - was it a specific design decision NOT to include an "address" field in location? All job forms I've ever filled out required a specific address. Also you might want to consider the equivalent of "zip code" (obviously something accepted more world-wide such as "post code").

I know this is probably WAY overcomplicating the schema - but have you considered creating a "desiredWorkLocations" key that could take options city/states/postcodes/rangeFrom (km), also "desiredWorkPositions" that could list specific job titles? This may help the job sites sort out what jobs to pair people with.

@osg
Copy link

osg commented Jul 9, 2014

There are several issues/thoughts in one:

  • Why is there no address field in location?
  • Consider using post[al] code [or postcode] rather than zip code.
  • Consider adding a desiredWorkLocations key.
  • Consider adding a desiredWorkPositions key.

For this reason, it is hard for this issue to be specifically actionable, and for it to gain traction. Also, the title street address? is vague.

Which issue/thought is most important to you?

@ocram
Copy link
Contributor

ocram commented Jul 9, 2014

According to the issue's title, the street thing was most important to @danvideo I guess.

I've added this as a more targeted issue here: #77

Regarding the desiredWorkLocations and desiredWorkPositions, I don't think that should be a part of the schema, in addition to these problems having no relation to this issue's title.

@DonDebonair
Copy link
Member

We shouldn't use street, but use address or addressline instead. In the context of internationalisation, it makes more sense. Also, +1 for post(al)code instead of zipcode.

@ocram
Copy link
Contributor

ocram commented Jul 11, 2014

"Postal code" is the most general and common writing: http://en.wikipedia.org/wiki/Postal_code

"Street" is probably too narrow, you're right.

@DonDebonair
Copy link
Member

Proposal: address for stuff like street and housenumber, postalCode for the postal code :)

@ocram
Copy link
Contributor

ocram commented Jul 11, 2014

@dandydev +1

@osg
Copy link

osg commented Jul 11, 2014

I suspect that an addressline1 (addressLine1?) and optional addressline2 (addressLine2?) could be useful. Nothing at the top of my head wrt resumes, but in Berlin I have to include that information for packages to get delivered. Apartments do not have numbers, and one must include a second address line, such as Hinterhaus 5. Etage li. In this case, if I am delivering mail, I need to go into the rear house (of the courtyard) to the 5th floor, and the apartment is on the left.

@DonDebonair
Copy link
Member

And couldn't you include that information in one field? Hinterhaus 5. Etage li could be one line, right?

@osg
Copy link

osg commented Jul 11, 2014

I have never seen it on one line. If it were to be one line, it would be between 41 - 48 characters in this case.

@DonDebonair
Copy link
Member

Seems a valid reason :)

Revised proposal: addressLine1, addressLine2, postalCode.

Please vote +1 if agree.

@osg
Copy link

osg commented Jul 11, 2014

+1

Happy to dogfood it soon. :)

@DonDebonair
Copy link
Member

I need to convince my wife to move to a fancier place that requires two addressLines ;)

@osg
Copy link

osg commented Jul 11, 2014

OT: Create another issue for that. :P

@ocram
Copy link
Contributor

ocram commented Jul 12, 2014

@opensourcegrrrl and @dandydev -1 on that. The specification should not be adjusted for styling purposes. What if someone else needs three lines then? You can just enter line breaks into the single address field.

@DonDebonair
Copy link
Member

But when you enter linebreaks into fields, you actually do include styling into your resume. There is precedent for having two address lines. [This article] does a nice explanation of different address formats in web forms, tailored to different national standards. Especially the chapter about Generic Formats towards the end of the article, seems to support our notion of having two address lines. Incidentally, it also deals with the Sate/Region problem (in a way that does not apply to our spec, but it's enlightening anyways).

In conclusion: I think there are enough scenarios in which one address line/street field isn't convenient for describing an address. Having two address lines solves that problem, and is an easy way of catering to many different address formats.

@osg
Copy link

osg commented Jul 12, 2014

@dandydev What was that link?

@thomasdavis
Copy link
Member

This issue has been elevated to decision needed

@ocram
Copy link
Contributor

ocram commented Jul 13, 2014

@dandydev No, if we add a separate addressLine2 field, we are including styling in the specification. However, if the user adds a line break to their single address field, they add styling to their own resume (or "implementation").

We should let the user enter whatever they want and, on our part, prefer not to include any styling/design issues in the specification, which should be more abstract and concentrate on the data.

The address is a single unit of information, so you should not split it up into several fields in the specification. By the way, your proposal of two address fields may not be enough, if that's the way we want to take this.

@osg
Copy link

osg commented Jul 13, 2014

+1 @mwaclawek's approach. Great clarification and reasoning, and it seems to solve my problem.

Next, how would the code comment read? How about this:

"To add an address line, use \n. For example, 1234 Glücklichkeit Straße\nHinterhaus 5. Etage li."

@DonDebonair
Copy link
Member

Sorry, this is the link: http://www.uxmatters.com/mt/archives/2008/06/international-address-fields-in-web-forms.php
I do think @mwaclawek has very good arguments though, so +1 for his approach of having one address field.

@DonDebonair
Copy link
Member

PR in #102

@DonDebonair
Copy link
Member

I botched up my PR, but @mwaclawek created a new one, which has now been merged.

@DonDebonair
Copy link
Member

Added to changelog.

@DonDebonair
Copy link
Member

I created an additional PR for clarifying how to add multiple address lines, as per @opensourcegrrrl 's example. Please review and merge :) #111

@thomasdavis
Copy link
Member

Great work guys and thanks @dandydev !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants