[openstreetmap/openstreetmap-website] GPX Import email in text format (PR #5031)

[Anton Khorev]

@AntonKhorev commented on this pull request.



> +      description_with_tags: "It looks like your GPX file %{trace_name} with the description %{trace_description} and the following tags: %{tags}"
       description_with_tags_html: "It looks like your GPX file %{trace_name} with the description %{trace_description} and the following tags: %{tags}"

The logic of this email, I assume, is not to tell the user how to download a file, but to tell which file was processed. The filename is likely generated by whatever track writing software and may not be easily recognizable, that's why we add an url. The user doesn't need to download the file because they've just uploaded it.

But if you think that your changes are easier to read, shouldn't the html version be also modified in this manner?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5031#discussion_r1698757582
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/5031/review/2210575710 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240731/a6c1cf13/attachment-0001.htm>