search

dronefly-garden / dronefly

Red Discord Bot V3 cogs for naturalists.
Other
16 stars 3 forks source link

search obs: search list timeout/delete #116

Closed synrg closed 4 years ago

synrg commented 4 years ago

I think the search list produced by [p]search obs should be removed if the result was that the user displayed at least one observation from it.

scenario 1:

,s obs something ... nothing selected, timeout after 30 seconds, search list reactions are removed automatically & search list remains.

scenario 2:

,s obs something ... user selects an observation, user presses :x: on observation, observation is removed, search list reactions timeout, search list remains (maybe? or is this too complex? Alternatively, treat this the same as the next scenario, i.e. remove search list because the user made a choice from it)

scenario 3:

,s obs something ... user selects an observation, user presses :white_check_mark: on observation, observation remains, search list times out, search list is removed.

scenario 4:

,s obs something ... user selects an observation, user presses :white_check_mark: on observation, observation remains, before timeout on search list the user presses :white_check_mark: on search list, all reactions are removed and search list remains.

synrg commented 4 years ago

Maybe it would be easier to implement and for the user to understand if the search list just always gets removed after the timeout unless they press :white_check_mark: to keep it, across all 4 scenarios.

synrg commented 4 years ago

Went a different way with this. No timeout. The user just chooses which they want to keep with :white_check_mark: or discard with :x: