[openstreetmap/openstreetmap-website] Notifier message pluralization (#2431)

Andy Allan notifications at github.com
Wed Nov 13 11:01:29 UTC 2019


Thanks for the PR @rubynho !

There is one change required to your PR. The travis build reports this error, from our `erblint` checks:

```
Style/HashSyntax: Use hash rockets syntax.
In file: app/views/notifier/gpx_success.html.erb:5
```

This means use e.g. `:trace_points => @trace_points` and not e.g. `trace_points: @trace_points`

If you can, please make this change, squash your commits and force-push to your branch. This will ensure there is only one commit in the history for this PR.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/2431#issuecomment-553353038
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20191113/2caa66e2/attachment.html>


More information about the rails-dev mailing list