Signalen / backend

Backend for Signalen, an application that helps cities manage and prioritize nuisance reports.
https://signalen.org
Mozilla Public License 2.0
5 stars 5 forks source link

Signal 568 #131

Open erikveerm opened 3 years ago

erikveerm commented 3 years ago

Could you take a look at signal 568: https://meldingen.s-hertogenbosch.nl/manage/incident/568

Signal made by Customer Service Afv:

Signal made by Customer Service Afv • Text notification classifies on category Other • Additional questions filled in of category Container • Routing to department AOG (correctly based on category Container/additional questions)
• List Assigned to from department AOG • Category Other

I cannot reproduce this on acceptance. Whatever I do, without adjusting the category to Container, I don't see the additional questions of Container

bartjkdp commented 3 years ago

I am unable to reproduce this problem as well. My guess would be a race condition in the frontend. @boris-arkenaar any thoughts?

boris-arkenaar commented 3 years ago

I don't understand the problem and I don't have credentials to log in to the 's-Hertogenbosch environment.

boris-arkenaar commented 3 years ago

Steps to reproduce:

  1. Log in https://s-hertogenbosch.signalen.dev/manage
  2. Go to 'Melden' https://s-hertogenbosch.signalen.dev/incident/beschrijf
  3. Select a source
  4. Click a point on the map
  5. Select subcategory 'Afval en vervuiling' -> 'Container'
  6. Type in 'Troep' in the textarea 'Waar gaat het om?'
  7. Select 'Nu'
  8. (You can notice the subcategory has switched to 'overig' -> 'overig')
  9. Click on 'Volgende'
  10. (You can notice there are two additional questions in the next step)
  11. Answer the questions
  12. Go through the rest of the form
  13. Submit the incident
CBuiVNG commented 3 years ago

While debugging this issue, I noticed that the sub cat was not always imported correctly. This PR https://github.com/Amsterdam/signals/pull/749 corrects that part in the back-end