[openstreetmap/openstreetmap-website] Fix handling of title when replying to diary entries (#1816)

Andy Allan notifications at github.com
Wed Apr 11 06:19:11 UTC 2018


I noticed that the link "Reply to this entry" includes the title for the message, but this wasn't being shown in the new message form. This PR includes a test to ensure that the form field is filled in correctly, and includes some extras to make the tests easier to write (taken from the moderation branch).
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/1816

-- Commit Summary --

  * Fix handling of title when replying to diary entries

-- File Changes --

    M app/views/diary_entry/_diary_entry.html.erb (2)
    M app/views/message/new.html.erb (4)
    M config/routes.rb (2)
    A test/system/diary_entry_test.rb (18)
    M test/test_helper.rb (8)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/1816.patch
https://github.com/openstreetmap/openstreetmap-website/pull/1816.diff

-- 
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/1816
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20180410/571f81b7/attachment.html>


More information about the rails-dev mailing list