topcoderinc / va-pseudovet

pseudovet Project For VA
1 stars 1 forks source link

'Add Diagnosis' list won't populate #16

Closed sdgun closed 6 years ago

sdgun commented 6 years ago

'Add Diagnosis' list won't populate

Target URL

http://127.0.0.1:5000/configuration/create

Steps to Reproduce

  1. Open the URL http://127.0.0.1:5000/
  2. Click 'Create configuration' Tab
  3. Fill all the details and go to 'RELATED CONDITIONS' section
  4. Expand 'Add Diagnosis' drop down

Actual Result(s)

Expected Result(s)

Frequency

Environment Details

Image(s)/Video(s)/Console Log/Crash Log

8

lijulat commented 6 years ago

@talesforce @dustinWeaver Can you give more info on how this field should work?. This was out of scope from the last challenge.

lijulat commented 6 years ago

@dustinWeaver Can you confirm this as well? What data should be in this field and also how does this field affect the dataset generation

talesforce commented 6 years ago

@dustinWeaver - Please confirm on this

dustinWeaver commented 6 years ago

Is the issue that the user can't search for a new morbidity? If so, the user should be able to search for one

dustinWeaver commented 6 years ago

When a user enters additional morbidities here, the data generation should include these as well

lijulat commented 6 years ago

@dustinWeaver This is ‘Related Diagnosis’ section. Currently there is no info on the what to be shown in this field . And how this field effects dataset generation

dustinWeaver commented 6 years ago

It's behavior should be just like the diagnosis part. This just allows a user to enter additional related diagnosis

lijulat commented 6 years ago

@dustinWeaver does that mean this field will be a user input and not a list to choose from?

ShankarKamble commented 6 years ago

@dustinWeaver @talesforce : Can you please confirm , 'RELATED CONDITIONS' section is in scope or not. If yes what will be values for that image

As per defect description 'Add Diagnosis' should have populate in 'RELATED CONDITIONS' section.

lijulat commented 6 years ago

@talesforce Can we get clarity on this soon ?

talesforce commented 6 years ago

does that mean this field will be a user input and not a list to choose from? -> Yes

As per defect description 'Add Diagnosis' should have populate in 'RELATED CONDITIONS' section. -> Please pick dummy values for now, make them configurable so we can change it later as needed

lijulat commented 6 years ago

@talesforce Your reply is actually contradicting the field referred does that mean this field will be a user input and not a list to choose from? and As per defect description 'Add Diagnosis' should have populate in 'RELATED CONDITIONS' section. both are same.

dustinWeaver commented 6 years ago

The list for the related diagnosis should be the same as the main diagnosis page for now. We'll actually need to have a separate list.

It's user input based on the list