-
Notifications
You must be signed in to change notification settings - Fork 4
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
Emojis in async results threads; allow setting packet name for bulk playtesting; bulk playtesting reacts per-packet indexing; auto-add users to thread based on category roles; fix various other issues #38
Open
ani-per
wants to merge
57
commits into
JemCasey:main
Choose a base branch
from
ani-per:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ting packet number/name
…kflow and prevent overlap of async/bulk/echo channels; improve button constructor
…tion number detection
…de` as dependency for `initDB`
…ted category role
* Create functions for removing and extracting question number * Create function for adding users with given role to given thread * Prevent thread titles from exceeding the limit * Reorganize / tidy up results thread for tossups * Add `ANS:` emoji
* Add functions to tally/summarize reacts for bulk playtested questions * Add bulk playtested questions table to database * Properly award powers if question is powermarked * Add `!end` command to finish packet reading * Remove echo setting and make concise echo the default
…from original repo)
…d title to fix issue JemCasey#19; clean qb meta language from thread titles; use subbullets in async tossup results
This was referenced Dec 27, 2024
…n Thread" button
Updated main post to reflect fixing of #18. |
…ssues in spoilerfree elements
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See the updated README for more clarifications about each of these new features.
React emojis
See the new reacts zip file.
Configuration
The configuration process is now proofed against channels being duplicated for asynchronous and bulk playtesting (i.e. no async channels can be bulk or echo channels, and vice versa).
Auto-adding category roles
Users are now auto-added to all bot-created threads based on their category roles (
RMPSS
,Literature
, etc.).Powermarking
Powermarked questions are now properly parsed and players are given 15 points for buzzing in at any clue phrase before the powermark is read.
Asynchronous playtesting
Question
Discussion Thread Button
There's now a "Create Discussion Thread" button that allows anyone to trigger the creation a thread if
!t
wasn't used in the question to auto-create the thread.Results
Tossups
Bonuses
For pseudo-tabular display, the emojis are now placed first, followed immediately by the score and user.
Bulk playtesting
Packet name
Users can now set the packet name for bulk playtesting. See the updated README for more details.
Reacts
Indexing (echoing)
React tallying
Using the
!tally A
or!count A
command will automatically tally the reacts for all questions in packet A and publish the tallies to the corresponding index entry in the echo channel. Using the!end
command will automatically tally the reacts for all questions in whatever packet was being read until that point.Issues Fixed
!t
/!thread
in separate message to create thread for the most recent question #18u
to go back to the previous clue on tossups and back to the previous part on bonuses (except for after directing final bonus part)!config
are incorrect in terms of spacing #29