Skip to content
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

Lemmy game threads #25

Open
wants to merge 27 commits into
base: lemmy-mlb-game-threads
Choose a base branch
from

Conversation

benja810
Copy link

Context

I created a new bot for game threads in lemmy. The idea is to provide the same game day thread experience that exists currently on the sports subreddits but on lemmy. I am attempting to start with baseball and move on from there.

Notes

I copied the existing game_threads bot and all of its templates. I also commented out sticky functionality for the moment and weekly threads.

In order to get things working, I need to rip out all the praw references and replace them with my own "plaw" version. This isn't a real praw for lemmy, it's just enough to get the game day bot working minimally.

Known Bugs

The Game Day Thread loop creates a title from its template that has a line break, which isn't allowed in lemmy (API returns an error).

@benja810
Copy link
Author

benja810 commented Jun 15, 2023

TODO

My current list

  • Create new config for lemmy bot user
  • Fix Game Thread multi-line title issue
  • Fix whatever issue is preventing the threads from being updated once they start

@toddrob99 toddrob99 changed the base branch from master to lemmy-mlb-game-threads June 20, 2023 19:17
@toddrob99 toddrob99 added the enhancement New feature or request label Jun 20, 2023
@@ -445,4 +445,15 @@ def upgrade_database(toVer=None):
time.time()
),
],
14: [
# Add Sidebar Updater bot type
Copy link
Owner

Choose a reason for hiding this comment

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

Nice, you saved me the step of doing this... It looks like you forgot to update the comment after copy/pasting though.

Update __init__.py to fix time check for post game thread
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants