[openstreetmap/openstreetmap-website] Adding map note tags - part 1 - added migration script and model files (PR #5323)

Nenad Vujicic notifications at github.com
Thu Nov 14 14:40:23 UTC 2024


<!--
Please read the contributing guidelines before making a PR:
  https://github.com/openstreetmap/openstreetmap-website/blob/master/CONTRIBUTING.md

Pay particular attention to the section on how to present PRs:
  https://github.com/openstreetmap/openstreetmap-website/blob/master/CONTRIBUTING.md#pull-requests
-->

### Description
<!--Describe your changes in detail. If you have made changes to the UI, include screenshots. If your PR addresses a Github issue, please link to it.-->

Added migration for creating `note_tags` DB table, created model file `NoteTag.rb`, updated associations with `Note` class and added `note_tag` factory and unit tests `NoteTagTests` for testing basic functionalities (key/value lengths validity, key/value lengths invalidity, orphaned tags invalidity and note-tags uniqueness).

This PR is first step of adding support for map note tags described in #5294 and which fully implementation can be found [here](https://github.com/nenad-vujicic/openstreetmap-website/tree/issue_5294_poc) (decomposed to smaller because of simpler review).

### How has this been tested?
<!--Explain the steps you took to test your code.-->

Tested by running linters from `lint.yml` workflow, unit tests and by manually inserting note-tags manipulations and as part of [complete implementation](https://github.com/nenad-vujicic/openstreetmap-website/tree/issue_5294_poc) locally.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Added NoteTag model and note_tags table
  * Added note_tag factory and NoteTag model test-case

-- File Changes --

    M app/models/note.rb (2)
    A app/models/note_tag.rb (20)
    A db/migrate/20241030122707_create_note_tags.rb (13)
    M db/structure.sql (35)
    A test/factories/note_tags.rb (8)
    A test/fixtures/note_tags.yml (11)
    A test/models/note_tag_test.rb (49)

-- Patch Links --

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

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5323
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/5323 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20241114/66a41e19/attachment.htm>


More information about the rails-dev mailing list