[openstreetmap/openstreetmap-website] Communities - communities (PR #3717)

Brian DeRocher notifications at github.com
Sat Sep 24 19:50:23 UTC 2022


Microcosms was renamed to communities  See https://github.com/openstreetmap/openstreetmap-website/pull/3683 for the PR before this one.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Add microcosm model and controller (show, show_by_key) and tests.
  * Create the edit handler, but it doesn't do anything yet.  Guests may not edit.
  * Guest can also show_by_key.
  * Remove trailing space.
  * Add a list of all miccrocosms.
  * bundle install friendly_id for slugs.
  * Replace show_by_key with using friendly_id slugs.
  * Convert Facebook and Twitter links into has_many links.
  * Add location to microcosms.
  * Admins can create and edit microcosms.
  * Move the "new" microcosm link to the header.
  * Microcosm row layout
  * Remove empty files.
  * Removed disabled code.
  * When the creation save fails, send the user back to the form.
  * StrongMigrations is a module.
  * erblint
  * Forgot to commit this file.
  * Set not null on microcosm.{description,slug} and link attributes.
  * Move styles to stylesheets.
  * Remove disabled code.
  * Replace link to geofabrik with TODO item.
  * rubocop
  * Use save not save!.
  * find_or_initialize! should be find_or_initialize.
  * Add missing NOT NULL constraints FK constraints.
  * Translate all the things.
  * Correcting a mistake.  This table should have been created with min/max lat/lon as integer.
  * Change lat and lon from decimal to integer.
  * Replace hardcoded image of map with a leaflet map.
  * Replace .flex_row with bootsrap row which is also flex-based.
  * erblint fix
  * Header should contain the name of the microcosm.
  * Rename lat to latitude, lon to longitude, convert all coords to floats.
  * Replace hard coded changesets with real ones.
  * Add some validation and model testing for microcosms.
  * Rename microcosm_valid() to validate() in order to bring out the similarity of this method.
  * whitespace
  * Add validation and model test for microcosm_link.
  * Add validation for microcosm locations.
  * Microcosm descriptions are not allowed to be nil/empty.
  * Add missing entry for microcosms in compact-secondary-nav
  * allow anyone to create a microcosm.  For now?
  * rubocop got pissy about indentation
  * Move microcosm :new, :create to non-admin.
  * Add instruction, all fields are required.
  * add hint for location
  * Order microcosms by longitude.
  * Remove inline javascript to comply with CSP.  Move that logic to microcosms.js.
  * Add a bunch of tests for microcosm_controller.
  * test microcosm update
  * Add more tests.  I prefer @title so templates don't need to call functions like t().
  * This test can now run as plain user not administrator.
  * Make the forms look like the rest of the website.
  * Make microcosm index look better.  Move edit link
  * Factor out showMap code which will be used on multiple pages.
  * Modify UI for mobile.
  * Factor out formMapInput.
  * Display if the changeset has review_requested.
  * Sort by longitude and time of day
  * Rubocop said replace content_tag with tag.
  * Use tag (in place of content_tag) correctly.
  * Don't pollute the global namespace.
  * Subject: [PATCH 208/280] Improve error messages for microcosm controller and
  * Add tests for controller when failures happen.
  * Adapt to rubocop.  In tests factor out some WET code into helper methods.
  * Format these coords better.
  * Rubocop cleanups
  * Subject: [PATCH 237/280] Normalize longitude input before using it.
  * Add an organizer to the microcosm.
  * Allow users to report a microcosm.
  * Get tests working by not using :controller/:action in tests.
  * Should hav added organizer_id a while ago.
  * Show the microcosms a user is an organizer of.
  * Merge remote-tracking branch 'upstream/master' into microcosms-microcosms
  * Non-admins can create and edit their own microcosms.
  * Use declarative spec for who can edit microcosms.
  * Add CRUD and testing for MicrocosmLinks.
  * Resolve erblint and brakeman findings.
  * prefer string interpolation
  * Use bootstrap styles for the list of links.
  * Use .content_map from common.
  * Use the same design pattern as in diary entries for secondary-actions.
  * Use nested resources.
  * Move the sorting logic for microcosms to a separate function, fix it, and document it.
  * Rename microcosms_i_organize to microcosms_organized.
  * Move microcosms of_user into microcosms#index.
  * Get these tests passing once again now that we're using nested resource routes.
  * Move normalize_longitude from controller to a shared function in lib/osm.rb.  Use it at the model level.
  * Use translation for helper text in forms.
  * Remove unused file.
  * RFC 6761 - reserved domain names
  * No need for assert_no_missing_translations in check_page_basics.
  * Consolidate db migrations.
  * No need to ignore a column ("key") that is no longer there.
  * Rename Microcosms to Communities.
  * Merge remote-tracking branch 'upstream/master' into communities-communities

-- File Changes --

    M Gemfile (4)
    M Gemfile.lock (7)
    M app/abilities/ability.rb (5)
    M app/assets/javascripts/application.js (74)
    A app/assets/javascripts/communities.js (18)
    A app/assets/stylesheets/communities.scss (9)
    A app/controllers/communities_controller.rb (102)
    A app/controllers/community_links_controller.rb (59)
    M app/controllers/issues_controller.rb (2)
    M app/controllers/reports_controller.rb (2)
    M app/helpers/issues_helper.rb (4)
    M app/models/client_application.rb (1)
    A app/models/community.rb (61)
    A app/models/community_link.rb (21)
    M app/models/issue.rb (2)
    M app/models/report.rb (1)
    M app/models/user.rb (2)
    A app/views/communities/_form.html.erb (30)
    A app/views/communities/_index_list.html.erb (25)
    A app/views/communities/edit.html.erb (3)
    A app/views/communities/index.html.erb (18)
    A app/views/communities/new.html.erb (5)
    A app/views/communities/show.html.erb (75)
    A app/views/community_links/_form.html.erb (5)
    A app/views/community_links/edit.html.erb (3)
    A app/views/community_links/index.html.erb (27)
    A app/views/community_links/new.html.erb (5)
    M app/views/layouts/_header.html.erb (4)
    M app/views/users/show.html.erb (6)
    A config/initializers/friendly_id.rb (107)
    M config/locales/en.yml (86)
    M config/routes.rb (11)
    A db/migrate/20210517030520_create_communities.rb (19)
    A db/migrate/20220820220545_create_friendly_id_slugs.rb (14)
    A db/migrate/20220821143545_create_community_links.rb (11)
    M db/structure.sql (220)
    M lib/osm.rb (5)
    M test/abilities/abilities_test.rb (21)
    A test/controllers/communities_controller_test.rb (277)
    A test/controllers/community_links_controller_test.rb (204)
    A test/factories/communities.rb (14)
    A test/factories/community_links.rb (7)
    A test/models/community_link_test.rb (21)
    A test/models/community_test.rb (86)
    M test/models/issue_test.rb (5)
    A test/system/report_community_test.rb (33)
    M test/test_helper.rb (8)

-- Patch Links --

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

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

Message ID: <openstreetmap/openstreetmap-website/pull/3717 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20220924/02dceac4/attachment-0001.htm>


More information about the rails-dev mailing list