Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

context changes #308

Closed
wants to merge 10 commits into from
Closed

context changes #308

wants to merge 10 commits into from

Conversation

noamoss
Copy link
Contributor

@noamoss noamoss commented May 8, 2020

fixes #307
fixes #290
fixes #245
fixes #331

update chatbot to current context

@noamoss noamoss requested a review from mushon May 8, 2020 08:38
@noamoss
Copy link
Contributor Author

noamoss commented May 8, 2020

@mushon please review and offer textual / flow changes

@akariv not sure why, on local, every time I get to the insulation block, the bot stops. Can you take a look and suggest/fix?

This was referenced May 8, 2020
steps:
- switch:
arg: sex
match: female
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No cases...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed, and still, something here breaks the script

@@ -657,7 +681,7 @@
cases:
- default: true
steps:
- say: האם ביקור בבית תפילה הוא חלק מהשיגרה שלך?
- say: האם ביקרת בבית תפילה הוא חלק מהשיגרה שלך?
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

לא עברית...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

אם משנים את הנוסח, צריך לוודא שהשאלה נשאלת פעם בשבוע

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@akariv
תיקנתי את הנוסח, טפל בבקשה אתה בנושא המופע השבועי
.

scripts/script.yaml Outdated Show resolved Hide resolved
scripts/script.yaml Outdated Show resolved Hide resolved
scripts/script.yaml Outdated Show resolved Hide resolved
options:
- show: "כן, ידוע לי מספר החולה"
value: "yes"
- show: אני בבידוד כי נחשפתי לחולה מאומת
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

משהו פה השתבש לך

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

לא מבין את המשוב...

scripts/script.yaml Outdated Show resolved Hide resolved
scripts/script.yaml Outdated Show resolved Hide resolved
scripts/script.yaml Outdated Show resolved Hide resolved
scripts/script.yaml Outdated Show resolved Hide resolved
- show: במקום אחר
class: other
value: "none"

- show: לא, אין לי קורונה
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

אם בוודאות אין לי קורונה, למה שאהיה בבידוד?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

אולי - ״אין לי קורונה (כנראה)״

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

אם בוודאות אין לי קורונה, למה שאהיה בבידוד?

  • מרצוני החופשי
  • פגשתי נשא
  • אחד מבני ביתי חולה ואני מטפל/ת בו.

בכל אופן, אני חושב שהפתרון המוצע ב-#324

יתן מענה גם לזה. מציע שקודם נטפל בנושא שם, וכאן נדאג רק להתאמות הנוסחים

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?אם פגשתי נשא או שאחד מבני ביתי חולה, איך אני יכול לומר שבוודאות אין לי קורונה

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

זה יכול להיות (בודאות קטאם נבדקת היום/אתמול וקיבלת תשובה שלילית בבדיקה
זו לא וודאות מדעית, אלא המוודעת/הבנה של המדווח/ת את מצבם.

ושוב, מציע שנסגור קודם את #324, ואז אולי דברים יסתדרו גם כאן.

@@ -681,7 +690,7 @@
cases:
- default: true
steps:
- say: האם ביקרת בבית תפילה הוא חלק מהשיגרה שלך?
- say: האם ביקרת בבית תפילה כחלק מהשיגרה השבוע?
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

אפשר להוריד את ״כחלק מהשיגרה״

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

מתלבט...
הרעיון היה להפריד בין ביקור חד פעמי בבית התפילה לבין ביקורים חוזרים

@@ -497,8 +500,12 @@
value: hotel
- show: בבית
value: home
- show: החלמתי ואני בבית
value: recovered
- show: החלמתי, אבל אני עוד בבית
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

אם החלמתי אז למה אמרתי שאני חולה בקורונה?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

צודק, אבל מציע שקודם נטפל ב-#324,
שם כבר לזה מענה

- show: במקום אחר
class: other
value: "none"

- show: לא, אין לי קורונה
value: none
steps:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

שואלים פה על בידוד או אשפוז אבל אין התייחסות לאשפוז באפשרויות עצמן

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

שוב, מציע שנסכם קודם את #324,
ואז נעשה את את ההתאמות הנדרשות בנוסחים - אם יהיה צורך.

@akariv akariv closed this Jun 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants