casa icon indicating copy to clipboard operation
casa copied to clipboard

assigned volunteers has a has many through to active case assignments…

Open svileshina opened this issue 3 months ago • 2 comments

What github issue is this PR for, if any?

Resolves #6496

What changed, and why?

added a comment with explanation

How is this tested? (please write rspec and jest tests!) 💖💪

Note: if you see a flake in your test build in github actions, please post in slack #casa "Flaky test: " :) 💪 Note: We love capybara tests! If you are writing both haml/js and ruby, please try to test your work with tests at every level including system tests like https://github.com/rubyforgood/casa/tree/main/spec/system

Screenshots please :)

Run your local server and take a screenshot of your work! Try to include the URL of the page as well as the contents of the page.

Feelings gif (optional)

What gif best describes your feeling working on this issue? https://giphy.com/ How to embed: ![alt text](https://media.giphy.com/media/1nP7ThJFes5pgXKUNf/giphy.gif)

svileshina avatar Sep 13 '25 17:09 svileshina

I agree with the chosen resolution for now, that we are going to let the N+1 issue remain and add a comment, cuz there is not a clear technical solution for this and the UX for this page would probably be overhauled anyways so lets not optimize what may change soon anyways.

Since this PR is a draft, leaving open for now.

gvt avatar Sep 13 '25 18:09 gvt

This PR has been open for a long time without any pushes or comments! What's up?

github-actions[bot] avatar Dec 10 '25 02:12 github-actions[bot]