Skip to content
This repository has been archived by the owner on Sep 14, 2024. It is now read-only.

Stop Minerva from sending two initializes for the same event #39

Open
chrissank opened this issue Sep 11, 2021 · 2 comments
Open

Stop Minerva from sending two initializes for the same event #39

chrissank opened this issue Sep 11, 2021 · 2 comments
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@chrissank
Copy link
Collaborator

image

Sometimes after a /initialize Minerva sends two initialize messages for the same event (denoted by having the same names). Only the first (i.e. the event that occurs the earliest) should be used.

@chrissank chrissank added the good first issue Good for newcomers label Sep 11, 2021
@chrissank
Copy link
Collaborator Author

this happens when there are less than 10 (or something) events that have undefined descriptions over the next week, so it starts to grab the next event in the repeated event as well

@QuantumManiac QuantumManiac added the bug Something isn't working label Apr 4, 2023
@QuantumManiac
Copy link
Member

Bug still exists, reproduced in dev:

  1. Create a weekly repeating event in the calendar, with it being the only event in the calendar
  2. Invoke /initialize in any of the channels in slack
  3. you will see an initialization message for the event 10 times:
    image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants