Skip to content

Commit

Permalink
areas.txt has oddly named "id" field instead of "area_id"
Browse files Browse the repository at this point in the history
Suggested change to name of field to make it fit other such fields in
the spec.
  • Loading branch information
thomastrillium committed May 8, 2017
1 parent 12c7eb8 commit caeb8bf
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion spec/reference.md
Expand Up @@ -27,7 +27,7 @@ A key concept in flexible services is the idea of a service area or zone. These

| Field Name | Required? | Details |
|------------|------------|---------|
| id | Required | The **id** field contains an ID that uniquely identifies an area. |
| area_id | Required | The **id** field contains an ID that uniquely identifies an area. |
| lat | Required | The **lat** field specifies the latitude of a single point in the area’s polygon. The field value must be a valid WGS84 latitude. |
| lon | Required | The **lon** field specifies the longitude of a single point in the area’s polygon. The field value must be a valid WGS84 longitude. |
| sequence | Required | The **sequence** field associates the latitude and longitude of a single point with its sequence order in the area’s polygon. The value of **sequence** must be a non-negative integers and must increase sequentially between each point in the polygon. |
Expand Down

2 comments on commit caeb8bf

@tsherlockcraig
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@antrim @barbeau being a github noob, I thought my commit was going to initiate a PR, but, being a contributor, I guess I can just change the spec.

I feel confident in the change, but let me know if this was decided in the past for a reason.

@barbeau
Copy link
Member

@barbeau barbeau commented on caeb8bf May 8, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

Please sign in to comment.