Allow property lookup by slug

Author: etewiah

app/controllers/api_public/v1/properties_controller.rb

@@ -7,7 +7,9 @@ def show
         locale = params[:locale] || I18n.default_locale
         I18n.locale = locale
         # Use listed_properties (materialized view) instead of deprecated props
-        property = Pwb::Current.website.listed_properties.find(params[:id])
+        scope = Pwb::Current.website.listed_properties
+        property = scope.find_by(slug: params[:id]) || scope.find_by(id: params[:id])
+        raise ActiveRecord::RecordNotFound unless property
         render json: property.as_json
       rescue ActiveRecord::RecordNotFound
         render json: { error: "Property not found" }, status: :not_found

spec/requests/api_public/v1/properties_spec.rb

@@ -21,6 +21,14 @@
       expect(json["id"]).to eq(realty_asset.id)
     end
 
+    it "returns the property by slug" do
+      host! 'properties-test.example.com'
+      get "/api_public/v1/properties/#{realty_asset.slug}"
+      expect(response).to have_http_status(200)
+      json = response.parsed_body
+      expect(json["id"]).to eq(realty_asset.id)
+    end
+
     it "returns 404 for non-existent property" do
       host! 'properties-test.example.com'
       get "/api_public/v1/properties/00000000-0000-0000-0000-000000000000"