cityofaustin / atd-mobile-signals-work-orders

Mobile Signals Work Orders app using React UI with Knack API
https://mobile-signals.austintexas.io
Other
3 stars 1 forks source link

Update Update Inventory Table for New Technician Workflow #242

Closed mddilley closed 4 years ago

mddilley commented 4 years ago

Closes https://github.com/cityofaustin/atd-data-tech/issues/1212 Closes https://github.com/cityofaustin/atd-data-tech/issues/1232

This PR updates the scene and view for the Knack API request that populates the Inventory accordion. Additional fields are displayed as well along with rows being highlighted based on the value of the Status field. This also updates the field used for the Work Order Details title (intersection) to fix a bug where it was missing in the view.

The formatting of the fields in the Inventory table may need to be adjusted but I'm kicking that down the road to address in https://github.com/cityofaustin/atd-data-tech/issues/1215 and https://github.com/cityofaustin/atd-data-tech/issues/1216.

Inventory accordion and updated work order location header

image

mateoclarke commented 4 years ago

I already clicked approved, and I don't think this is something that needs to be fixed on the code side, but maybe in the Knack staging instance.

As test-tech@ & mateo.clarke@ I'm having issues requesting from a specific view: https://us-api.knack.com/v1/scenes/scene_297/views/view_889/connections/field_513?rows_per_page=2000

Getting 403 Forbidden. Is there an attribute I need to update on either of those accounts?

Screen Shot 2020-01-28 at 9 21 25 AM
mddilley commented 4 years ago

Awesome catch - I did not notice this when working on this branch. The view_id for the form for adding inventory items changed in Knack so this is updated in PR #243. https://github.com/cityofaustin/atd-mobile-signals-work-orders/blob/c402c7af3dc2f088858af692d02c132710999ca3/src/queries/api.js#L85