[openstreetmap/openstreetmap-website] Diary entry comment subscriptions (#1309)

Mikel Maron notifications at github.com
Tue Oct 4 02:29:11 UTC 2016


This PR implements subscription to diary entry comments. Subscribe/unsubscribe button placed below comment box. 

A few issues remain before this is totally ready to go, could use help on the issue with tests
* https://github.com/mikelmaron/openstreetmap-website/issues/5
* https://github.com/mikelmaron/openstreetmap-website/issues/4
* https://github.com/mikelmaron/openstreetmap-website/issues/6

Also not sure about placement of button at bottom of the comments box.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * WIP diary comment subscriptions
  * not api endpoints, moved to button, fixed notifier message, fixed tests

-- File Changes --

    M app/controllers/diary_entry_controller.rb (43)
    M app/models/diary_entry.rb (1)
    M app/models/notifier.rb (8)
    M app/models/user.rb (1)
    M app/views/diary_entry/_diary_entry.html.erb (1)
    M app/views/diary_entry/view.html.erb (5)
    M config/locales/en.yml (24)
    M config/routes.rb (2)
    A db/migrate/20161002153425_add_join_table_between_users_and_diary_entries.rb (23)
    M test/controllers/diary_entry_controller_test.rb (97)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/1309.patch
https://github.com/openstreetmap/openstreetmap-website/pull/1309.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/1309
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20161003/fdfc85da/attachment.html>


More information about the rails-dev mailing list