[openstreetmap/openstreetmap-website] Adds optional use of notes records (PR #5511)
Nenad Vujicic
notifications at github.com
Fri Jan 17 12:09:51 UTC 2025
### Description
PR adds use of `description`, `user_id`, `user_ip` from `notes` table if data-migration is done, otherwise use `body`, `author_id`, `author_ip` from note's first comment. Decision procedure "data-migration is done" is defined with "is user_ip or user_id not nil?".
This PR is 4th from set of PRs described [here](https://github.com/openstreetmap/openstreetmap-website/pull/5485#pullrequestreview-2545467592).
### How has this been tested?
Automated unit tests and manual testing.
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/5511
-- Commit Summary --
* Adds optional use of notes records
-- File Changes --
M app/models/note.rb (27)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/5511.patch
https://github.com/openstreetmap/openstreetmap-website/pull/5511.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5511
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/5511 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250117/ad1de17f/attachment.htm>
More information about the rails-dev
mailing list