cssat / sprout-issues

An issue-only repository for the Sprout data collection platform
https://sproutservices.org
2 stars 1 forks source link

Locking down moving reports from other referrals/family #211

Closed tinhart closed 4 months ago

tinhart commented 11 months ago

Describe the bug 8/16 - A referral needs to be entered on the referral it was intended for and not moved after creation. Is there another approach? Not sure this will get the wanted results (people will still be able to do it, just won't be able to fix it). Prevent provider from creating a visit report on a closed referral? Reports are being moved from family to family across cases - Sarah confirmed this is possible. Update CR: If move a visit report, can only move it within the same case.

To Reproduce Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior A referral needs to be entered on the referral it was intended for and not moved after creation.

Screenshots If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context 10/10 - Exploration. Made some progress. Reviewed a development plan. Signed off on approach. No code changes or testing (needed?). Not sure if this means the issue is resolved/complete...

keithligon commented 10 months ago

Code out for review: https://github.com/cssat/sprout/pull/158