-
Notifications
You must be signed in to change notification settings - Fork 16
nextmap overhaul #162
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
base: master
Are you sure you want to change the base?
nextmap overhaul #162
Conversation
|
@BalaM314 not quite sure what's with all the github actions bots, but they seem pretty broken rn |
|
mildly concerned about throwback maps cuz i have slight trauma from seeing maps like frontline be played 5 million times and i do not want those times to come back |
|
i meant you need to update the pnpm-lock.yaml, not the package-lock.json... but you don't have PNPM I should make the actions bot do it also this pr needs rebasing |
e932947 to
9748247
Compare
c6d6776 to
975703a
Compare
c7fd8f1 to
7510eca
Compare
| showVotes(); | ||
| } | ||
|
|
||
| args.map ??= await currentMenu(sender); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have a function that fills in the value of an omitted argument of type map... I think this should be moved into the commands framework
…sh-commands into feat-maps_menu
Features
/nextmapnow includes a no-arg option, which displays a advanced map-selecting GUI (see pictures). GUI additionally features space for future throwback map functionality (Throwback Maps #155) and campaign support (Proper Campaign Support #156)/mapscommand depreciatedImages