Skip to content

Add location mapping and address display to Community Fridges - #534

Closed
neal-bpm wants to merge 4 commits into
vn/community-fridge-photo-upload-v1from
vn/community-fridge-location-mapping-v1
Closed

neal-bpm wants to merge 4 commits into
vn/community-fridge-photo-upload-v1from
vn/community-fridge-location-mapping-v1

Conversation

@neal-bpm

@neal-bpm neal-bpm commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Describe your changes

Wires the Community Fridges UI to real map coordinates. The index page gains a Leaflet map with pins for every fridge that has coordinates, an Address column in the table, and a LiveLocation field in the form for setting/editing a fridge's location. The data layer is refactored to use cast_assoc for nested location params (replacing the bare location_id foreign key), and all context functions now preload :location.

Checklist before requesting a review

  • I have performed a self-review of my code
  • If it is a core feature, I have added tests.
  • Are there other PRs or Issues that I should link to here?
  • Will this be part of a product update? If yes, please write one phrase
    about this update in the description above.

…integration

- Preload location data in community fridge queries and forms.
- Update changeset to handle nested location parameters.
- Add map display and address column in community fridge index.
- Improve tests to validate location handling and map rendering.
@coderabbitai

coderabbitai Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 54 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: bikebrigade/dispatch/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 64559ec1-1907-497a-97ff-a2cfc8dcf57d
📥 Commits

Reviewing files that changed from the base of the PR and between aab451b and e31545b.

📒 Files selected for processing (9)
  • lib/bike_brigade/locations.ex
  • lib/bike_brigade/locations/community_fridge.ex
  • lib/bike_brigade_web/live/community_fridge_live/form_component.ex
  • lib/bike_brigade_web/live/community_fridge_live/form_component.html.heex
  • lib/bike_brigade_web/live/community_fridge_live/index.ex
  • lib/bike_brigade_web/live/community_fridge_live/index.html.heex
  • test/bike_brigade/locations/community_fridge_test.exs
  • test/bike_brigade_web/live/community_fridge_live_test.exs
  • test/support/fixtures.ex
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@neal-bpm
neal-bpm marked this pull request as ready for review October 8, 2026 12:12
@neal-bpm neal-bpm changed the title Vn/community fridge location mapping v1 Add location mapping and address display to Community Fridges Oct 9, 2026
@neal-bpm

neal-bpm commented Oct 9, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #540. Work consolidated into vn/community-fridge-feature for unified review.

@neal-bpm neal-bpm closed this Oct 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant