Skip to content

Add ?lqvote and ?votequeue for flavortown#176

Open
alexlam0206 wants to merge 10 commits intohackclub:mainfrom
alexlam0206:ft-macros-nok
Open

Add ?lqvote and ?votequeue for flavortown#176
alexlam0206 wants to merge 10 commits intohackclub:mainfrom
alexlam0206:ft-macros-nok

Conversation

@alexlam0206
Copy link
Copy Markdown

@alexlam0206 alexlam0206 commented Mar 12, 2026

?lqvotes / ?lqvote

Hey (user), some of your votes are likely being marked as low quality votes and are discarded.
Go to voting. You will get a project shown to you. Make sure you are:

  • checking both the repo and demo links
  • spending some time looking through the project's devlogs
  • giving good, thoughtful scores and feedback
  • generally caring and not rushing through voting
    Low quality votes won't count towards your vote requirement!

?votequeue

Hey (user), we currently have a backlog of projects waiting for votes to be voted on. Please be patient.

I've marked this question as resolved, so please start a new thread if it's too long or you need more help!

Copilot AI review requested due to automatic review settings March 12, 2026 12:45
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds two new Flavortown-specific Slack macros to help support helpers respond to common voting-related questions (vote backlog + low-quality vote warnings) by extending the transcript configuration and introducing new macro handlers.

Changes:

  • Add votequeue_macro and lqvotes_macro to the base Transcript model and implement their Flavortown transcript messages.
  • Introduce two new macros: ?votequeue / ?voteq and ?lqvote / ?lqvotes.
  • Document the new macros in README.md.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
nephthys/transcripts/transcripts/flavortown.py Defines the new Flavortown transcript messages used by the new macros.
nephthys/transcripts/transcript.py Adds new optional transcript fields for vote queue + low-quality vote messaging.
nephthys/macros/votequeue.py Implements the ?votequeue macro behavior (reply + resolve).
nephthys/macros/lqvotes.py Implements the ?lqvote(s) macro behavior (reply + resolve).
README.md Lists the new macros in the documentation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Copilot AI review requested due to automatic review settings March 12, 2026 13:07
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Copilot AI review requested due to automatic review settings March 25, 2026 11:49
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@MMK21Hub
Copy link
Copy Markdown
Collaborator

I've made quite a few changes to your PR - hope that's okay!

I've removed votequeue, updated the wording of lqvotes, and changed it from being called lq_votes to low_quality_votes in the code

Also note that new macros must also be added to the macro_list. I've done that for you!

image image

@MMK21Hub
Copy link
Copy Markdown
Collaborator

Just wanted to check that you're happy with the current state of the PR before I merge it? Since it has your name on it

@alexlam0206
Copy link
Copy Markdown
Author

Ohh okay thanks for editing it :thumbsup-dino:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants