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.

Sign up for GitHub

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

Jump to bottom

✨ Query - Show calendar for Brendan #224

Closed
1 task done
jimmidier opened this issue Nov 3, 2021 · 4 comments · Fixed by #264, #283 or #291
Closed
1 task done

✨ Query - Show calendar for Brendan #224

jimmidier opened this issue Nov 3, 2021 · 4 comments · Fixed by #264, #283 or #291
Assignees
@AttackOnMorty
Milestone
v1.0.0

Comments

@jimmidier
Copy link
Collaborator

jimmidier commented Nov 3, 2021 •
edited by AttackOnMorty
Loading

Cc: @jernejk @AttackOnMorty

Pain

From @jernejk's feedback,

What is Jason's agenda?

We need to support asking someone's agenda.
Related to #139 (comment)

Tasks

  • Query -
    • "What is Jason's agenda?"
    • "Show Jason's Bookings"
    • "What's Jean's next booking?"

Screenshots

image
Figure: Refer to response from the old bot

Thanks!
@jimmidier jimmidier added this to the v1.0.0 milestone Nov 3, 2021
@jimmidier jimmidier self-assigned this Nov 3, 2021
@jimmidier jimmidier mentioned this issue Nov 11, 2021
Merged
@AttackOnMorty
Copy link
Member

AttackOnMorty commented Nov 12, 2021 •
edited
Loading

  • Change card title to Patrick Zhao’s next appointments [10]
  • Add icon before the name and button
  • Change “Details in CRM” to “More Booking Info”

@AttackOnMorty AttackOnMorty reopened this Nov 12, 2021
@AttackOnMorty AttackOnMorty self-assigned this Nov 15, 2021
@AttackOnMorty AttackOnMorty mentioned this issue Nov 16, 2021
Merged
@jimmidier jimmidier reopened this Nov 18, 2021
@jimmidier jimmidier mentioned this issue Nov 18, 2021
Merged
@AttackOnMorty
Copy link
Member

AttackOnMorty commented Nov 19, 2021 •
edited by jimmidier
Loading

  • Data is not correct. Calum doesn't have any appointments on Sunday.

image
Figure: Calendar shows Calum have an appointment on Sunday

image
Figure: PeopleAPI data source

@AttackOnMorty AttackOnMorty reopened this Nov 19, 2021
@AttackOnMorty
Copy link
Member

AttackOnMorty commented Nov 19, 2021 •
edited
Loading

  • TBD: Current calendar shows the first appointment of the day. One person might have multiple appointments in a day. Should it show all the appointments? (we can't get full outlook calendar, only CRM booking)

image
Figure: Bot response

@jimmidier
Copy link
Collaborator Author

jimmidier commented Nov 22, 2021 •
edited
Loading

  • Data is not correct. Calum doesn't have any appointments on Sunday.

image Figure: Calendar shows Calum have an appointment on Sunday

image Figure: PeopleAPI data source

image
Figure: Fixed by #290

@AttackOnMorty AttackOnMorty mentioned this issue Nov 25, 2021
Closed
1 task
@AttackOnMorty AttackOnMorty changed the title Query - Show calendar for Brendan ✨ Query - Show calendar for Brendan Nov 25, 2021
@jimmidier jimmidier assigned AttackOnMorty and unassigned jimmidier Nov 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AttackOnMorty AttackOnMorty

Labels
None yet
Projects
None yet
Milestone
v1.0.0
2 participants
@jimmidier @AttackOnMorty