hasadna / reportit-agent

1 stars 1 forks source link

fix 'should you update a variable' quetsions on replay #65

Closed noamoss closed 4 years ago

noamoss commented 4 years ago

tried on https://admin.equality.org.il/#/report/185.

  1. I started to answer a report (initial intake). entered some info (like the involved company name).
  2. clicked back to the main dashboard. The example report appeared there with 'בטיפול' status + 2 open tasks,
  3. I clicked on it, the conversation ran from the beginning (not as a replay).
  4. I went back to the main dashboard, and clicked again on the same report.. The same happened - I had to answer the questions to continue, so I clicked on the first two buttons to move on. Then I clicked on the menu button to go back to the main dashboard, again
  5. clicked on the same report again, and this time, the display stopped where I got in phase 4 (i.e. waiting for my 3rd answer). As you can see on the screenshot my original company name was saved, but I am being asked again, instead of just showing be the last call

image

it might be worth to mention this check was made right after using the script editor for the first time, implementing the required change on #50.

akariv commented 4 years ago

@noamoss did you try this after fixing the bug in the script?

noamoss commented 4 years ago

@akariv just did, so now it works as expected in terms of not asking for the input again, but it runs gradually, and not like it was supposed to according to #64 - the thread runs again, in a slow phase, instead of showing it all at once

(https://admin.equality.org.il/#/report/187): image

noamoss commented 4 years ago

(you might want to close this issue, in favor of #64, fine by me)

noamoss commented 4 years ago

@akariv reopenning this issue, as a similar problem appeared:

  1. after answering 'no' to a question: image

  2. I click back to the dashboard, and enter the report again, the replay works until the same question: image

so now we are not talking about the fact it does not follow #64. Not sure if helps, but my guess is the problem occurs because we are still using a function to ask and save the response of the agent, which prevent you from replaying it.

akariv commented 4 years ago

Yes, replay doesn't happen for questions initiated from inside functions. Once you finish getting rid of them, please reopen if there's still a problem.

On Mon, Dec 16, 2019, 00:20 noamoss notifications@github.com wrote:

@akariv https://github.com/akariv reopenning this issue, as a similar problem appeared:

1.

after answering 'no' to a a question: [image: image] https://user-images.githubusercontent.com/3581741/70870239-6ac56480-1f99-11ea-944e-8f449547a7fe.png 2.

I click back to the dashboard, and enter the report again, the replay works until the same question: [image: image] https://user-images.githubusercontent.com/3581741/70870254-9ba59980-1f99-11ea-96a2-2bc97579bad3.png

so now we are not talking about the fact it does not follow #64 https://github.com/hasadna/reportit-agent/issues/64. Not sure if helps, but my guess is the problem occurs because we are still using a function to ask and save the response of the agent, which prevent you from replaying it.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/hasadna/reportit-agent/issues/65?email_source=notifications&email_token=AACAY5KWIPGC42IBD3SN3OLQY2UUDA5CNFSM4J2WMQDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEG5DWWQ#issuecomment-565853018, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACAY5MJPXY53XXICCWOZTTQY2UUDANCNFSM4J2WMQDA .

noamoss commented 4 years ago

@akariv blocked by https://github.com/hasadna/reportit-agent/issues/70 please reassign to me when ready for check

noamoss commented 4 years ago

the main problem here is solved, but a minor side effect was found, to be reported in a separate issue: https://github.com/hasadna/reportit-agent/issues/74