Spike of community resource browsing

CHANGELOG.md

@@ -12,6 +12,10 @@ We're still in the [initial development phase](https://www.jering.tech/articles/
 This changelog also serves to acknowledge the incredible people who've contributed brilliance, effort and being. Their handles are listed under the first release they each  touched. 💗🙏🏾
 
 ## [Unreleased]
+
+### Enhancements
+* Show Community Resources in the same contributions page as all the other contributions
+
 ### Security vulnerabilities fixed
 * Fix bug in authorization logic that allows a guest user to view names, emails, addresses of other people #951
     * We recommend all installations update to this release IMMEDIATELY.

app/blueprints/contribution_blueprint.rb

@@ -3,9 +3,9 @@
 class ContributionBlueprint < Blueprinter::Base
   identifier :id
   association :categories_for_tags, name: :category_tags, blueprint: DefaultBlueprint
-  association :service_area, blueprint: ServiceAreaBlueprint, view: :with_location
+  association :service_areas, blueprint: ServiceAreaBlueprint, view: :with_location
   association :contact_types, blueprint: DefaultBlueprint do |contribution, _options|
-    [contribution.person.preferred_contact_method]
+    [contribution.preferred_contact_method]
   end
   association :urgency, blueprint: DefaultBlueprint do |contribution, _options|
     UrgencyLevel.find(contribution.urgency_level_id)

app/filters/contact_method_filter.rb

@@ -8,6 +8,10 @@ def self.filter_grouping
 
   def filter(scope)
     return super unless parameters
+
+    # We may want to add contact method logic for community resources. For now, this effectively allows all
+    return super unless scope.respond_to? :person_id
+
     scope.joins(:person).where(people: {preferred_contact_method: parameters.keys})
   end
 end

app/filters/contribution_type_filter.rb

@@ -2,10 +2,11 @@ class ContributionTypeFilter < BaseFilter
   def self.filter_grouping
     {name: 'Contribution Types', filter_options: [
       {id: 'ContributionType[Ask]', name: 'Ask'},
-      {id: 'ContributionType[Offer]', name: 'Offer'}
+      {id: 'ContributionType[Offer]', name: 'Offer'},
+      {id: 'ContributionType[CommunityResource]', name: 'Community Resource'}
     ]}
   end
-  ALL_ALLOWED_TYPES = ['Ask', 'Offer'].freeze
+  ALL_ALLOWED_TYPES = %w[Ask Offer CommunityResource].freeze
 
   def filter(scope)
     raise NotImplementedError.new(

app/filters/service_area_filter.rb

@@ -8,6 +8,6 @@ def self.filter_grouping
 
   def filter(scope)
     return super unless parameters
-    scope.where(service_area_id: parameters.keys)
+    scope.in_service_areas(parameters.keys)
   end
 end

app/javascript/pages/browse/ListBrowser.vue

@@ -23,7 +23,11 @@
             {{ contribution.urgency.name }}
           </b-tag>
         </td>
-        <td>{{ contribution.service_area.name }}</td>
+        <td>
+          <div v-for="service_area in contribution.service_areas" :key="service_area.id" class="has-text-grey-lighter">
+            {{ service_area.name }}
+          </div>
+        </td>
         <td style="text: nowrap;">
           <SingleIcon :iconType="contribution.contact_types[0].name" />
         </td>
@@ -32,7 +36,7 @@
             <a :href="contribution.view_path" class="button icon-list is-primary"><span class=""> View</span></a>
           </div>
         </td>
-        <td v-if="showContributorNames">{{ contribution.person.name }}</td>
+        <td v-if="showContributorNames ">{{ !!contribution.person ? contribution.person.name : ' ' }}</td>
       </tr>
     </table>
   </div>
@@ -55,7 +59,7 @@ export default {
   },
   computed: {
     showContributorNames() {
-      return this.contributions.some(contribution => contribution.person.name)
+      return this.contributions.some(contribution => contribution.person && contribution.person.name)
     },
   },
 }

app/javascript/pages/browse/MapBrowser.vue

@@ -36,6 +36,11 @@
       filters: Object,
     },
     components: { Mapbox },
+    computed: {
+      mappableContributions() {
+        return this.contributions.filter((c) => { return c.location || c.service_areas && c.service_areas.length > 0 })
+      }
+    },
     methods: {
       loaded(map) {
         let geojson = {
@@ -47,9 +52,9 @@
           return new Promise((resolve) => {
             let forwarded = 0
 
-            this.contributions.forEach(contribution => {
+            this.mappableContributions.forEach(contribution => {
               geoCodingService.forwardGeocode({
-                query: `${contribution.location && contribution.location.street_address ? contribution.location.street_address : ""} ${contribution.service_area.location.city} ${contribution.service_area.location.state} ${contribution.service_area.location.zip_code}`,
+                query: `${contribution.location && contribution.location.street_address ? contribution.location.street_address : ""} ${contribution.service_areas[0].location.city} ${contribution.service_areas[0].location.state} ${contribution.service_areas[0].location.zip}`,
                 limit: 1
               }).send()
                 .then(response => {

app/javascript/pages/browse/TileListItem.vue

@@ -29,10 +29,10 @@
           </div>
           <h5 class="has-text-weight-bold">{{ title }}</h5>
           <p>{{ description }}</p>
-          <div v-if="service_area" class="has-text-grey-lighter">
+          <div v-for="service_area in service_areas" :key="service_area.id" class="has-text-grey-lighter">
             {{ service_area.name }}
           </div>
-          <div v-if="person.name" class="contributor-name">
+          <div v-if="person && person.name" class="contributor-name">
             {{ `From: ${person.name}` }}
           </div>
         </div>
@@ -55,7 +55,7 @@ export default {
   props: {
     contribution_type: String,
     category_tags: {type: Array, default: () => []},
-    service_area: {type: Object, default: null},
+    service_areas: {type: Array, default: () => []},
     title: String,
     inexhaustible: Boolean,
     description: String,

app/models/community_resource.rb

@@ -17,10 +17,17 @@ class CommunityResource < ApplicationRecord
 
   validates :name, :description, :publish_from, presence: true
 
+  # We're not sure if we need or want these validations. Adding them now assuming it will be easier to remove them later than the other way around
+  validates :service_areas, presence: true
+  validates_associated :service_areas
+  validates :tag_list, presence: true
+
   accepts_nested_attributes_for :organization
 
   scope :approved,       -> { where(is_approved: true) }
   scope :pending_review, -> { where(is_approved: false) }
+  # TODO: add tests for this?
+  scope :in_service_areas, ->(ids) { joins(:service_areas).where(service_areas: {id: ids}).distinct }
 
   def self.published
     before_now = DateTime.new..Time.current
@@ -31,20 +38,48 @@ def self.published
     )
   end
 
+  def title; description; end
+  def self.matchable; published; end
+
   def published?
     now = Time.current
     is_approved &&
       (publish_from.present? ? publish_from <= now : true) &&
       (publish_until.nil? || now < publish_until)
   end
 
+  def categories_for_tags
+    Category.where(name: tag_list)
+  end
+
   def all_tags_unique
     all_tags_list.flatten.map(&:downcase).uniq
   end
 
   def all_tags_to_s
     all_tags_unique.join(', ')
   end
+
+  def preferred_contact_method
+    # TODO: This is a hack that makes things work for now
+    # The test creates a contact method that will match this
+    # and the dev db seeding creates a couple, too
+    ContactMethod.method_name('call').last
+  end
+
+  def type
+    "Community Resource"
+  end
+
+  def inexhaustible
+    true
+  end
+
+  def urgency_level_id
+    UrgencyLevel::TYPES.last.id
+  end
+
+  def person; end
 end
 
 # == Schema Information

app/models/listing.rb

@@ -33,6 +33,7 @@ class Listing < ApplicationRecord
   scope :match_status, ->(match_status) { where(state: match_status.to_s) }
   scope :person_id, ->(person_id) { where(person_id: person_id.to_i) }
 
+  scope :in_service_areas, ->(ids) { where(service_area_id: ids) }
   scope :service_area_name, ->(service_area_name) {
     includes(service_area: :mobility_string_translations)
       .references(:mobility_string_translations)
@@ -112,9 +113,17 @@ def categories_for_tags
     Category.where(name: tag_list)
   end
 
+  def service_areas
+    [service_area]
+  end
+
   def has_email?
     person.email.present?
   end
+
+  def preferred_contact_method
+    person.preferred_contact_method
+  end
 end
 
 # == Schema Information

spec/blueprints/contribution_blueprint_spec.rb

@@ -44,7 +44,7 @@
           'name' => contribution.person.preferred_contact_method.name
         }
       },
-      'service_area' => {
+      'service_areas' => [{
         'description' => contribution.service_area.description,
         'id' => contribution.service_area.id,
         'name' => expected_area_name,
@@ -58,7 +58,7 @@
           'street_address' => contribution.service_area.location.street_address,
           'zip' => contribution.service_area.location.zip
         }
-      },
+      }],
       # "map_location" => "44.5,-85.1",
       'title' => contribution.title,
       'description' => contribution.description,
@@ -73,4 +73,29 @@
     result = ContributionBlueprint.render(contribution, show_view_path: true, current_user: user)
     expect(JSON.parse(result)['view_path']).to eq(expected_path)
   end
+
+  it 'can serialize a community resource as a contribution' do
+    resource = create(:community_resource)
+    # The test database defaults to having no contact methods, so we need at least one
+    default_contact_method = create(:contact_method)
+    expected_result = {
+      "id" => resource.id,
+      "category_tags" => [],
+      "contact_types" => [{"id" => default_contact_method.id, "name" => "Call"}],
+      "contribution_type" => "Community Resource",
+      "created_at" => resource.created_at.to_f * 1000,
+      "description" => "Food for the revolution",
+      "inexhaustible" => true,
+      "location" => nil,
+      "match_path" => nil,
+      "name" => "Free breakfast for School Children Program",
+      "person" => nil,
+      "service_areas" => [],
+      "title" => "Food for the revolution",
+      "urgency" => {"id" => 4, "name" => "Anytime"},
+      "view_path" => nil
+    }
+    result = ContributionBlueprint.render(resource, current_user: user)
+    expect(JSON.parse(result)).to eq(expected_result)
+  end
 end

spec/factories/community_resources.rb

@@ -3,6 +3,8 @@
     name { 'Free breakfast for School Children Program' }
     description { 'Food for the revolution' }
     publish_from { Date.current }
+    service_areas { [association(:service_area)] }
+    tag_list { [association(:category).name] }
   end
 end
 

spec/models/community_resource_spec.rb

@@ -1,6 +1,6 @@
 require 'rails_helper'
 
-RSpec.describe CommunityResource do
+RSpec.describe CommunityResource, type: :model do
   describe "community resource can have multiple service areas" do
     it "has many service areas" do
       should respond_to(:service_areas)
@@ -9,4 +9,21 @@
       should_not respond_to(:service_area)
     end
   end
+
+  describe "validation" do
+    it "can be valid" do
+      expect(build(:community_resource)).to be_valid
+    end
+
+    it "is invalid if missing a service area" do
+      subject = build(:community_resource)
+      subject.service_areas = []
+      expect(subject).to_not be_valid
+    end
+
+    it "is invalid if missing a tag" do
+      subject.tag_list = []
+      expect(subject).to_not be_valid
+    end
+  end
 end