<p dir="ltr">Thanks a lot for the great feedback, Peter! I will incorporate it in my next round of improvements for sure.</p>
<p dir="ltr">Martijn</p>
<div class="gmail_quote">On Oct 31, 2014 4:19 PM, "Peter Barth" <<a href="mailto:osm-peda@won2.de">osm-peda@won2.de</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Martijn,<br>
<br>
Martijn van Exel schrieb:<br>
> If you already have gone through the challenge creation process, I would<br>
> really appreciate it if you could give me feedback on this.<br>
<br>
I have read the tutorial and I think it would have helped me :) I have<br>
some random comments, perhaps some can be used to improve the tutorial<br>
even further. Anyway, I like it already as is.<br>
<br>
* It's in the curl line but remention that one needs to set the<br>
  Content-Type of the http header to application/json<br>
<br>
* It's perhaps not obvious to everyone, why you can use the osmid in<br>
  GeoJSON's properties without a leading 'n' or 'w' (or 'r'). You should<br>
  note that this is dependent on the actual geometry type.<br>
<br>
* If you upload the challenge, you need POST, you named that. However,<br>
  it might not be obvious that you need PUT for updates. This might be<br>
  confusing as you don't see the challenge without tasks. So one might<br>
  wrongly assume he still has to POST to create the challenge<br>
<br>
* Sanity checks: You did miss to upload tasks. Only the challenge was<br>
  uploaded and thus it will not be visible at all. Btw, this should be<br>
  noted, too: Challenges are only visible as long as there are open<br>
  tasks (perhaps in the Maintainance section)<br>
* Also note that one can activate/deactivate tasks (e.g. emergency<br>
  disabling the challenge due to a task upload bug)<br>
<br>
* I'd note the 5000 task limit (even though it's in the API doc)<br>
<br>
And finally some notes about the API docs that got me "confused".<br>
<br>
* Bulk create/update of tasks can change the status, too. One can upload<br>
  JSON that *only* contains identifier and status.<br>
<br>
* What are valid values for status to be sent to the server. Afair<br>
  there's a mismatch of values returned and values that can be set or<br>
  something like that.<br>
<br>
* And the difference between 'available', 'created', 'editing' and<br>
  'skipped' might not be obvious. Especially which of these have to be<br>
  "maintained" to be available again and which ones will be made<br>
  available automatically.<br>
<br>
Hope some of that helps. And while much of it is already explained in<br>
the API docs, I think redundancy helps to get things right.<br>
<br>
Regards,<br>
Peda<br>
<br>
</blockquote></div>