[openstreetmap/openstreetmap-website] Add Communities page (#3301)

Adam Hoyle notifications at github.com
Mon Jul 18 21:59:57 UTC 2022


@atomoil commented on this pull request.



> @@ -0,0 +1,12 @@
+module OsmCommunityIndex
+  require "yaml"
+
+  def self.community_index
+    @community_index ||= community_index_from_json
+  end
+
+  def self.community_index_from_json
+    json_file = Rails.root.join("node_modules/osm-community-index/dist/resources.json")

Thanks for taking a look @openbrian (and sorry for the delay replying 😬). My intention with using Open Community Index was to follow the "single source of truth" principle. OCI already have the data we need to render the page on osm.org, so seems rude not to try to use it and share data across the OSM ecosystem. I do appreciate that a shorter-term solution of just hardcoding the data would have been easier to achieve, and we could always fall back to that if this approach doesn't pan out.

> Maybe OSM should be the system of record for communities.

Do you mean OCI or do you have something else in mind (if so I'd love to hear what you're thinking)?

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

Message ID: <openstreetmap/openstreetmap-website/pull/3301/review/1042602768 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20220718/dd4374e8/attachment.htm>


More information about the rails-dev mailing list