<p>In app/views/changeset/_changeset.xml.builder:</p>
<pre style='color:#555'>> @@ -0,0 +1,26 @@
> +changeset_attributes = { :id => changeset.id, :created_at => changeset.created_at.xmlschema, :closed_at => changeset.closed_at, :open => changeset.is_open? }
> +changeset_attributes[:uid] = changeset.user.id if changeset.user.data_public?
> +changeset_attributes[:user] = changeset.user.display_name if changeset.user.data_public?
> +changeset.bbox.to_unscaled.add_bounds_to(changeset_attributes, '_') if changeset.bbox.complete?
> +
> +xml.changeset(changeset_attributes) do |asterx|
> +  changeset.tags.each do |k,v|
> +    xml.tag :k => k, :v => v
> +  end
> +  if @comments
> +    xml.discussion do 
</pre>
<p>I would have implemented comments by passing the <code>:include_discussion</code> flag to <code>Changeset.to_xml</code> and <code>Changeset.to_xml_node</code>, but I don't know if that's the better solution <img class="emoji" title=":wink:" alt=":wink:" src="https://assets-cdn.github.com/images/icons/emoji/unicode/1f609.png" height="20" width="20" align="absmiddle">. Equally, just unit testing the output of the builder against <code>to_xml</code> would ensure we stay in sync. The other benefit of implementing it in the <code>to_xml</code> methods is that it would then be trivial to include comments in the controller's <code>:query</code> and update methods, if anyone wants that.</p>

<p>About the document type - sorry, I didn't explain what I meant very well. Looking more closely, I see you're using the <code>RichText</code> helper, but hard-coding the type to "text". I think it would be useful to allow choice of that format by the commenter - I certainly find Markdown useful for linking and so forth.</p>

<p>About the outdating - that seems fine for now, although I think it's likely to be requested pretty quickly post-release <img class="emoji" title=":smile:" alt=":smile:" src="https://assets-cdn.github.com/images/icons/emoji/unicode/1f604.png" height="20" width="20" align="absmiddle"></p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/openstreetmap/openstreetmap-website/pull/772/files#r15990043">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/1419053__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcyMzEyMDIxMiwiZGF0YSI6eyJpZCI6MzU4NjU3NjJ9fQ==--edeb7082cee7cba2e0b9ef482a42253d3f9ee81d.gif" width="1" /></p>