[openstreetmap/openstreetmap-website] Proposal for adding support for map notes tags (Issue #5294)
Anton Khorev
notifications at github.com
Fri Dec 13 14:01:04 UTC 2024
We have a so called *note comments* table:
```sql
CREATE TABLE public.note_comments (
id bigint NOT NULL,
note_id bigint NOT NULL,
visible boolean NOT NULL,
created_at timestamp without time zone NOT NULL,
author_ip inet,
author_id bigint,
body text, -- (a) not always a comment
event public.note_event_enum -- (b) not a comment
);
```
As you can see, it contains things that are *not* comments:
- *(a)*: `text` is a description in case of opening events
- *(b)*: `event` is not a comment of course, other comment tables don't have *events*
Here's the changeset comments table for comparison:
```sql
CREATE TABLE public.changeset_comments (
id integer NOT NULL,
changeset_id bigint NOT NULL,
author_id bigint NOT NULL,
body text NOT NULL,
created_at timestamp without time zone NOT NULL,
visible boolean NOT NULL
);
```
You can turn `note_comments` table into an actual comments table if you remove things that are not comments (opening event rows, rows without comment text, `event` column). But you can also turn it into a note version table if you remove other things (comment event rows), modify some (`text` to always contain the description) and add what's missing (version numbers).
If we replace `note_comments` with two tables as described above, we'll have a pure comments table and a versions table. Then we can add tags referencing the versions table.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/issues/5294#issuecomment-2541527509
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/issues/5294/2541527509 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20241213/9b54783e/attachment.htm>
More information about the rails-dev
mailing list