hospitalrun-frontend icon indicating copy to clipboard operation
hospitalrun-frontend copied to clipboard

New Imaging Request - required visit not enforced

Open blestab opened this issue 5 years ago • 7 comments

When requesting a new imaging, visit is required but a user can still save without choosing a visit. See https://github.com/HospitalRun/hospitalrun-frontend/pull/2552#pullrequestreview-576596693

🐛 Bug Report

When creating a new imaging request, a user should not be able to save without specifying a visit.

To Reproduce

Go to staging https://staging.hospitalrun.io/ Create a new patient and create one visit for this patient Create a new imaging request leaving the visit field empty Notice you are allowed to save even though the visit is marked as required

Expected behavior

User should not be able to save a New Imaging request without selecting a visit.

Reference

See how the Patient or Type field are handled and do the same

Your Environment

  • node version: 6,8,10
  • fastify version: >=1.0.0
  • os: Mac, Windows, Linux
  • any other relevant information

blestab avatar Jan 27 '21 08:01 blestab

@blestab Can I be assigned to this task?

SamuelQZQ avatar Jan 27 '21 08:01 SamuelQZQ

@blestab Can I be assigned to this task?

Thank you @SamuelQZQ, Let's get #2557 out first then will assign this to you once done.

blestab avatar Jan 27 '21 12:01 blestab

Hi, Sorry I'm getting in the way :)

"See how the Patient or Type field are handled and do the same"

In this case I don't think it's okay to manage them in the same way, with which search criteria would you do it? I don't think it would be good to even see them all. It would be necessary to decide with which criteria to show the visits.

alexma01 avatar Jan 27 '21 22:01 alexma01

Hi, Sorry I'm getting in the way :)

"See how the Patient or Type field are handled and do the same"

In this case I don't think it's okay to manage them in the same way, with which search criteria would you do it? I don't think it would be good to even see them all. It would be necessary to decide with which criteria to show the visits.

Thanks @alexma01, you're not in the way at all, all ideas are welcome; Why don't you hop onto our slack and tell us more about your thoughts about this

blestab avatar Jan 28 '21 09:01 blestab

Hi @blestab , Is there anything that I can help in this issue?

bernoraj avatar Feb 06 '21 10:02 bernoraj

@bernoraj currently I'm afraid it's stuck. if you want to take it over, you're welcome to it

matteovivona avatar Feb 09 '21 11:02 matteovivona

Sure @tehKapa , I would like to take it up.

bernoraj avatar Feb 10 '21 07:02 bernoraj