[openstreetmap/openstreetmap-website] Note: Add body & author (PR #4481)
Gregory Igelmund
notifications at github.com
Tue Apr 23 12:18:40 UTC 2024
@grekko pushed 16 commits.
0538035c7eccd5b4b8e9fff7e45eff9858d6cf31 Extend Note with body & author, update controllers & views
e822bcc501c35cefa9a2fd5fe00f4b546c047178 Update Tests to reflect Note refactoring
63d2053d65a2e7477b48ba6c5511cb1ea256c317 Add migration task for Notes
5a52a4b5a59d951f1acbd2af91dad973f6765768 Fix broken references to Factory-definitions
d8573166f621ae8ee2a1e6426c00da483dd4e52d 20240107144209_add_author_and_body_to_notes.rb: Remove data migration SQL
369bf907b9b65dc3c9002172a644265cca8f4f2b Notes#show: Redact Note#body for deleted authors
63a8982e8f0a3ffdea434fb6483a4ff73cbf7291 NotesController#show: Filter explicitly for opened-comment
c64c8df6e0a7972e1231a80e739f2ef82971ebc0 Notes: Avoid calls to opened_note_comment once body is migrated
bfac5a025d340b183d9543cf5fb57f09194ac3d5 Extract Note::MigrateOpenedComment
8de8670403078aeb88626c7c87270621ae32ed3a Delint app/helpers/note_helper.rb
5191679042478d8b334c98e7b36865ccf5b55e62 NoteHelper#note_body: Handle missing author, fix tests
6b90863de0596d23439ce222c40fb75f32b7275c api/notes/_note.rss.builder: Do not expose display name of deleted users
41201ed230af161f8459c04a1fb77557b0a3adfd Fix Note#build_comments_with_extra_open_comment
d4753470f9e77c6991bc332c4d66136a8feb8a44 api/notes_controller_test.rb: Harden test with more deterministic setup
6c37c9941ee7f7c2151c96915a57fa37ba43eed4 Api/Notes#feed: Include extra open comment just for migrated notes
adb3ac2c501a25163fa8a757770abc307297f6cb Api/Notes#feed: return only a max of result_limit NoteComment objects
--
View it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4481/files/d5feee19a9bcc2b472fa4123f0d6535139d59940..adb3ac2c501a25163fa8a757770abc307297f6cb
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/4481/push/18119970678 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240423/eb817fd1/attachment.htm>
More information about the rails-dev
mailing list