Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
68 changes: 59 additions & 9 deletions docs/quest-definition/example.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"version": "2.0.0",
"version": "3.0.0",
"elements": [
{
"element_type": "Sidewalks",
Expand Down Expand Up @@ -187,6 +187,53 @@
"choice_text": "No"
}
]
},
{
"quest_id": 10,
"quest_title": "What amenities are available along this sidewalk?",
"quest_description": "Select all amenities that are present along this sidewalk",
"quest_type": "MultipleChoice",
"quest_tag": "amenity",
"quest_answer_choices": [
{
"value": "benches",
"choice_text": "Benches",
"image_url": "http://some_url.com/image.jpg"
},
{
"value": "trash_bins",
"choice_text": "Trash bins",
"image_url": "http://some_url.com/image.jpg"
},
{
"value": "water_fountains",
"choice_text": "Water fountains",
"image_url": "http://some_url.com/image.jpg"
},
{
"value": "bike_racks",
"choice_text": "Bike racks",
"image_url": "http://some_url.com/image.jpg"
},
{
"value": "shade_structures",
"choice_text": "Shade structures",
"image_url": "http://some_url.com/image.jpg"
},
{
"value": "public_art",
"choice_text": "Public art",
"image_url": "http://some_url.com/image.jpg"
}
]
},
{
"quest_id": 11,
"quest_title": "Are there any other notable features or issues with this sidewalk?",
"quest_description": "Please describe any other notable features or issues with this sidewalk that were not covered in the previous questions.",
"quest_image_url": "http://some_url.com/image.jpg",
"quest_type": "TextEntry",
"quest_tag": "sidewalk_notes"
}
]
},
Expand Down Expand Up @@ -319,10 +366,7 @@
],
"quest_answer_dependency": {
"question_id": 4,
"required_value": [
"1",
"2"
]
"required_value": ["1", "2"]

Choose a reason for hiding this comment

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

As you get more into GitHub, here's a friendly PR etiquette tip: I'd avoid mixing cosmetic and content changes in the same PR. The noisy diffs may divert attention from the "meat" of the PR. Good PRs usually have a narrow focus/scope.

In this case, I'm also a little worried that someone may inadvertently reverse the changes with some automated formatting tool—some editors run these automatically these days.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks for the heads up and the best-practices tip :)
In this case, that's exactly what happened - ran the Prettier formatter in VS Code.

Choose a reason for hiding this comment

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

We may want to check in that Prettier config to this repo so tools don't fight with each other over formatting.

}
},
{
Expand Down Expand Up @@ -491,10 +535,16 @@
"image_url": "http://some_url.com/image.jpg"
}
],
"quest_answer_dependency": {
"question_id": 4,
"required_value": "yes"
}
"quest_answer_dependency": [
{
"question_id": 4,
"required_value": "yes"
},
{
"question_id": 5,
"required_value": "yes"
}
]
},
{
"quest_id": 7,
Expand Down
Loading