search

JMurk / Valve_Hydrant_Mobile_Issues

Water Valve Status Mobile Issues
0 stars 0 forks source link

Hydrant Status: photos associated to an inspection #51

Open rssidlowski opened 8 years ago

rssidlowski commented 8 years ago

I uploaded photos for a hydrant and they appear for both history records. See screen shots. photo1 photo2

Please investigate and see if the database model supports associating to an inspection and not the hydrant alone and pass to Lu if necessary.

JMurk commented 8 years ago

I'm not seeing the same thing. I know in the db the photos table stores both Hydrant ID and Inspection ID

rssidlowski commented 8 years ago

Upon further testing, I have two inspection records for this hydrant:

51_hydhist 51_hydstatus I added a photo to the 11/24/2014 inspection record. 51_addphoto

I save and return to the history and go to the other inspection (2/16) and click on the photo button and see the details for the photo that should be associated to the other inspection but no photo. 51_extraphoto

Expected behavior: The existing images screen should be blank if there are no photos for this inspection.

slmessier commented 8 years ago

Please reference the following graphic and implement the security constraints indicated for each of the 2 cases:

  1. Editor User case
  2. Read Only User case

image

yousifmalik commented 8 years ago

Completed and ready for testing on DEV

slmessier commented 8 years ago

I was able to successfully validate the full workflow using the DEV application. Issue will be marked as "Ready for Production".