Add a /dining soon subcommand #244
Labels
No labels
beginner friendly
bug
chore
documentation
duplicate
feature
help wanted
invalid
needs-triage
question
size/L
size/M
size/S
size/XL
size/XS
urgent
wontfix
needs/docs
needs/research
needs/upstream
type/bug
type/external
type/feature
type/refactor
urgency
high
urgency
immediate
urgency
low
urgency
medium
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
ScottyLabs/dalmatian#244
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Objective
Add
/dining soon, which returns a list of dining locations that are open now or opening in the next 60 mins. Essentially its output should only consist of locations marked with 🟢 (open), ⚠️ (open but closing soon) or 🔔 (closed but opening soon; this is what makes the command different from/dining open).Also add its
/dining soon-verboseequivalent, which does the same thing but with all page embeds sent at once.Suggested Approach
First, add the subcommands and their appropriate descriptions to the
commandvariable (Line 258 in src/commands/dining.ts). In the description, make sure to specify "open now or opening in the next 60 mins" (or equivalent). Don't just write "opening soon", since that's a little ambiguous.Then, take a look at Line 312 to see an example for
/dining open. You'll do the same thing but filter for open and opening soon locations.It may be helpful to check the conditional logic in
getCurrentStatusto see how we currently determine what status a location should get. Feel free to refactor the code andisOpenas needed./dining soonsubcommand #250