add prefix info to readme

all features are about done, let's goooooooooooooooooooo

.gitignore

@@ -9,6 +9,4 @@ htmlcov/
 
 dist/
 build/
-*.egg-info/
-
-config.py
+*.egg-info/

README.md

@@ -1,37 +1,97 @@
 # Tlapbot
-Tlapbot is an [Owncast](https://owncast.online/) bot, aiming to add the feature of channel points and
-channel point redeems to Owncast.
+Tlapbot is an [Owncast](https://owncast.online/) bot that adds channel points and
+channel point redeems to your Owncast page.
 
-This bot is currently in-development. The goal is to have a feature set on par
-with Twitch channel points, while making use of Owncast webhooks and especially
+This bot is currently in-development. The goal is to have an experience similar
+to [Twitch channel points](https://help.twitch.tv/s/article/viewer-channel-point-guide) by making use of [Owncast webhooks](https://owncast.online/thirdparty/webhooks/) and 
 [External actions](https://owncast.online/thirdparty/actions/).
 ## Features
-Currently, the bot gives points to everyone in chat -- the interval can be
-configured in the config, as well as the amount of points given.
+The bot gives points to everyone in chat -- 10 points every 10 minutes by
+default, but the time interval and amount of points can be changed.
 
-The users in chat can then use their points on redeems. The bot currently
-only has one hardcoded redeem, but I'd like to make this configurable,
-so that every Owncast streamer can set up their own redeems that best fit
-their stream.
+The users in chat can then use their points on redeems -- rewards like "choose my
+background music", "choose what level to play next", "react to this video" etc.
+You can configure redeems to fit your stream and the activities you're
+doing.
 
 The redeems then show on a "Redeems dashboard" that everyone can view
-at the flask server's URL, which can be included in Owncast
-as an External action, a single button that displays information about
-recent redeems.
+as an External Action on the Owncast stream, or at its standalone URL.
+This allows easy browsing of active challenges and recent redeems.
+### Tlapbot bot commands
+Tlapbot has these basic commands:
+- `!help` sends a help string in the chat, explaining how tlapbot works.
+- `!points` shows a chatter how many points they have.
+- `!name_update` is a special debug command, to be used with the user's name displays wrong in the redeem dashboard. Normally, it shouldn't have to be used at all, as display names get updated automatically when the bot is running.
+
+(If you change the default prefix of `!` to something else, these commands will
+use the new prefix instead.)
+
+Tlapbot also automatically adds a command for each redeem in the redeems file.
+### Tlapbot redeems types
+Tlapbot currently supports three different redeem types. Each type of a redeem
+works slightly differently, and displays differently on the redeems dashboard.
+
+Redeems can also optionally be sorted into "categories" that can be turned on
+or off in the config file. This means that the redeems file can list redeems
+for different types of streams, and you can turn them on or off. Examples on how
+to do that are listed later in the config file examples.
+#### List
+List redeems are basic redeems, most similar to the ones on Twitch.
+
+Every time a chatter redeems a List redeem, the redeem gets added to the list of recent redeems with a timestamp, similar to how redeems on Twitch get added to the [Twitch redeem queue](https://help.twitch.tv/s/article/making-the-most-of-channel-points?language=en_US#manage).
+
+Unlike the Note redeems, chatters can't write messages to send along with their List redeems, so make sure the redeem makes sense on its own, like "stop talking for one minute", or "drop your weapon", etc.
+#### Note
+Note redeems are like List redeems, they get added to the list of recent redeems.
+
+Unlike the List redeems, chatters can add a message to their Note redeems, so this is the ideal type for open-ended redeems like "choose what character I play as next", "choose what song to play next", etc.
+#### Counter
+Counter is a unique redeem type, in that it doesn't show up in the list of recent redeems, and counters don't list the people who redeemed them. This redeem type is good for any rewards or incentives where the important thing isn't "who redeemed it?" but rather "how many people redeemed it?"
+
+Instead, the tlapbot dashboard keeps a number for each "counter", and each redeems adds +1 to it.
+
+Counter redeems can be used to gauge interest, tally up votes, or to keep track of how many emotes should be added to an OBS scene.
+
+### Tlapbot dashboard
+Tlapbot dashboard is a standalone page available at `/dashboard`, made to be easily viewable as an owncast external action. The Tlapbot dashboard shows all redeems and active counters.
+
+Counters are at the top, followed by a chronological list of recent List and Note redeems.
+
+Tlapbot dashboard also shows the chatter's points balance when they open it as an external action.
+
+![Tlapbot dashboard](https://ak.kawen.space/media/67c1ac6ed0d2f4efb09937c1cbfe864102182e990796853507860f50db7ff5f5.png)
+
+*Tlapbot dashboard when viewed as an external action.*
+#### Redeems help tab
+The dashboard also has a "Redeems help" tab. It shows an explanation of redeem types,
+and lists all active redeems, along with their price, type and description.
+## License & Contributions
+Tlapbot as it currently is does not come with a license. If you're a content creator, streamer, vtuber, etc. I'll be happy to give you permission to use Tlapbot, or make changes that'd fit your stream.
+
+I didn't make Tlapbot available under a permissive or a free software license, as Owncast is also used by some religious groups, extremist individuals, and dubious corporations to self-host their streams, and I do not want for them to use the bot.
 ## Setup
+Tlapbot requires Python 3, probably a fairly recent version of it too. (My live instance runs on Python 3.9.2.)
+
 The Python prerequisites for running tlapbot are the libraries `flask`,
-`requests` and `apscheduler`.
-### Dev setup (from git repository)
+`requests` and `apscheduler`. If you follow the installation steps below,
+they should automatically be installed if you don't have them.
+
+You can also run Tlapbot in a [Python virtual environment](https://flask.palletsprojects.com/en/2.2.x/installation/#virtual-environments).
+### Install from git repo (as a folder)
 1. Clone the repository.
 2. Run `pip install -e .` in the root folder. This will install tlapbot
-as a package in editable more, along with all its prerequisites.
-3. Initialize db:
+as a package in editable more.
+3. Set the `FLASK_APP` environment variable.
+    ```bash
+    export FLASK_APP=tlapbot
+    ```
+4. Initialize the database:
     ```bash
     python -m flask init-db
     ```
-4. Create an `instance/config.py` file and fill it in as needed.
+5. Create a `instance/config.py` file and fill it in as needed.
 Default values are included in `tlapbot/default_config`, and values in
-`config.py` overwrite them. (The database also lives in the `instance` folder
+`config.py` overwrite them. (The database also lives in the instance folder
 by default.)
 
     Tlapbot might not work if you don't overwrite these:
@@ -40,35 +100,59 @@ by default.)
     OWNCAST_ACCESS_TOKEN # get one from owncast instance
     OWNCAST_INSTANCE_URL # default points to localhost owncast on default port
     ```
-## Owncast setup
-In Owncast, navigate to the admin interface at `/admin`,
+6. OPTIONAL: Create an `instance/redeems.py` file and add your custom redeems.  
+  If you don't add a redeems file, the bot will initialize the default redeems from `tlapbot/default_redeems.py`.  
+  More details on how to write the config and redeems files are written later in the readme.
+
+This installation is fine both for just running Tlapbot as it is, but it also works as a dev setup if you want to make changes or contribute.
+
+Updating should be as easy as `git pull`ing the new version.
+## Owncast  configuration
+In the Owncast web interface, navigate to the admin interface at `/admin`,
 and then go to Integrations.
 ### Access Token
-In Access Tokens, generate an Access Token to put in
-`instance/config.py`. The bot needs both the "send chat messages" and "perform administrative actions"
+In the Access Tokens tab, generate an Access Token to put in
+the config file in the instance folder. The bot needs both the "send chat messages" and "perform administrative actions"
 permissions, since getting the list of all connected chat users is an administrator-only
 action.
 ### Webhook
-In webhooks, create a Webhook, and point it at your bot's URL with
+In the webhooks tab, create a Webhook, and point it at your bot's URL with
 `/owncastWebhook` added.
 
-In debug, this will be something like `localhost:5000/owncastWebhook`,
-or, if you're not running the debug Owncast instance and bot on the same machine,
+In debug, this will be something like `localhost:5000/owncastWebhook`. If you're not running the debug Owncast instance and bot on the same machine,
 you can use a tool like [ngrok](https://ngrok.com/)
-to redirect the Owncast traffic to your `localhost`.
+to redirect Owncast traffic to your `localhost`.
 ### External Action
 In External Actions, point the external action to your bot's URL with `/dashboard` added.
 
-In debug, pointing the External Action to an address like `localhost:5000/dashboard` might not work because your localhost address doesn't provide https, which owncast requires.
+**External Actions only work with https. Your server will need to support SSL and
+https connections for this part to work.**
 
+In development, a `localhost` address will not work with External Actions, since it doesn't provide https.
 If you use [ngrok](https://ngrok.com/) to redirect Owncast traffic to localhost,
 it will work because the ngrok connection is https.
 
-**Example:**
+**External Action config example:**
 ```
-URL: MyTlapbotServer.com/dashboard
+URL: MyTlapbotServer.example/dashboard
 Action Title: Redeems Dashboard
 ```
+#### Note about https and reverse proxying
+Since External Actions require a secure https connection (for the tlapbot dashboard to work), you will need to set up a reverse proxy for tlapbot on your server. I'm not including much information about it here, since some knowledge of the topic is required to set up Owncast itself.
+
+The Owncast documentation about SSL and Reverse proxying is here: https://owncast.online/docs/sslproxies/
+
+If your followed the [Owncast recommendation to use Caddy](https://owncast.online/docs/sslproxies/caddy/) you'd only need to include this in your caddyfile to make the tlapbot dashboard work:
+
+```
+MyTlapbotServer.example {
+        reverse_proxy localhost:8000
+}
+```
+then MyTlapbotServer.example/owncastWebhook is the URL for webhooks,
+and MyTlapbotServer.example/dashboard is the URL for the dashboard.
+
+(And, obviously, you'd need to own the MyTlapbotServer.example domain, and have an A record pointing to your server's IP address.)
 ## Running the bot
 ### Running in debug:
 Set the FLASK_APP variable:
@@ -83,14 +167,120 @@ Run the app (in debug mode):
 ```bash
 python -m flask --debug run 
 ```
-### Running in prod:
-To be added when I actually run a prod version of the bot.
-## Config
-Values you can include in `instance/config.py` to change how the bot behaves.
-### Channel points interval and amount
-`POINTS_CYCLE_TIME` decides how often channel points are given to users in chat,
+### Running in production:
+Set the FLASK_APP variable:
+```bash
+export FLASK_APP=tlapbot
+```
+
+Using the flask debug server for running apps for non-development purposes is not recommended. Rather, you should be using a proper [Python WSGI server](https://flask.palletsprojects.com/en/2.2.x/deploying/).
+On my own live owncast instance, I use gunicorn.
+
+Install gunicorn (if you don't have it installed):
+```bash
+pip install gunicorn
+```
+Run the app (with gunicorn):
+```bash
+gunicorn -w 1 'tlapbot:create_app()'
+```
+
+You should also set `GUNICORN` in your `config.py` file to `True` to see Tlapbot logs in your gunicorn output. By default, gunicorn will only show `error` and `critical` warnings, but you can set the `--log-level` argument when running the app to set it to `warning`, `debug` or `info` too.
+
+```bash
+gunicorn -w 1 --log-level debug 'tlapbot:create_app()'
+```
+
+**⚠️WARNING:** Because of the way the scheduler is initialized in the project,
+I recommend running tlapbot with only one gunicorn worker. (`-w 1`)
+
+If you use multiple workers, each worker sets up its own scheduler, and then users
+get given points by each worker. (So running the app with `-w 4` means users get four times as many points than listed in the config.)
+
+I'd like to fix this shortcoming of tlapbot at some point in the future (so that it can take advantage of extra workers), but for now it's broken like this.
+
+## CLI commands: Updating redeems, clearing the queue
+Tlapbot comes with a few Click CLI commands. The commands let you clear out counters and the redeems dashboard.
+
+#### init-db
+The init-db command initializes the database.
+
+**This command should only be run when first installing tlapbot.**
+#### clear-queue
+The clear-queue command clears the redeem queue and resets all active counters to zero.
+You should run this command if you're about to start a new stream, and want to start with empty counters and queue.
+
+```bash
+python -m flask clear-queue
+```
+
+**If you've changed the config, and want new/different counters to work, you should run the `refresh-counters` command first.**
+#### refresh-counters
+This command deletes old counters that are no longer in the config file, and then adds all counters from the config file.
+You should run this command every time you've added or removed counters from `redeems.py`.
+```bash
+python -m flask refresh-counters
+```
+This command only changes counters, so if you want to clear the queue with list and note redeems too, you should run `clear-queue` after it, or run `clear-refresh` to do both actions together.
+
+#### clear-refresh
+Does the same as `clear-queue` and `refresh-counters` together.
+```bash
+python -m flask clear-refresh
+```
+Run this if you're adding/removing counters, want to reset them to zero and want to clear all redeems as well.
+## Configuration files
+### config.py
+Values you can include in `config.py` to change how the bot behaves.
+
+(`config.py` should be in the instance folder: `/instance/config.py` for folder install.)
+#### Mandatory
+Including these values is mandatory if you want tlapbot to work.
+- `SECRET_KEY` is your secret key. Get one from running `python -c 'import secrets; print(secrets.token_hex())'`
+- `OWNCAST_ACCESS_TOKEN` is the owncast access token that owncast will use to get list of users in chat. Generate one in your owncast instance.
+- `OWNCAST_INSTANCE_URL` is the full URL of your owncast instance, like `"http://MyTlapbotServer.example"`
+#### Optional
+Including these values will overwrite their defaults from `/tlapbot/default_config.py`.
+- `POINTS_CYCLE_TIME` decides how often channel points are given to users in chat,
 in seconds. 
+- `POINTS_AMOUNT_GIVEN` decides how many channel points users receive.
+- `LIST_REDEEMS` if `True`, all redeems will be listed after the `!help` command in chat.
+This makes the !help output quite long, so it's `False` by default.
+- `GUNICORN` if `True`, sets logging to use gunicorn's logger. Only set this to True if you're using Gunicorn to run tlapbot.
+- `ACTIVE_CATEGORIES` can be an empty list `[]`, or a list of strings of activated categories (i.e. `["chatting", "singing"]`). Redeems with a category included in the list will be active, redeems from other categories will not be active. Redeems with no category are always active.
+- `PREFIX` is a *single character string* that decides what character gets used as a prefix for tlapbot commands. (i.e. you can use `?` instead of `!`). Symbols are recommended. Prefix cannot be longer than one character.
+#### Example config:
+An example to show what your config like could look like
+```python
+SECRET_KEY= # string with secret key would be here.
+OWNCAST_ACCESS_TOKEN="5AT0gbe9ZuzDunsBG0rcwfalQNTi3fvV70NPvvQHk3I="
+OWNCAST_INSTANCE_URL="http://MyTlapbotServer.example"
+POINTS_CYCLE_TIME=300
+LIST_REDEEMS=True
+ACTIVE_CATEGORIES=["gaming"]
+```
+### redeems.py
+`redeems.py` is a file where you define all your custom redeems. Tlapbot will work without it, but it will load a few default, generic redeems from `tlapbot/default_redeems.py`.
 
-`POINTS_AMOUNT_GIVEN` decides how many channel points users receive.
+(`redeems.py` should be in the instance folder: `/instance/redeems.py` for folder install.)
+#### `default_redeems.py`:
+```python
+REDEEMS={
+    "hydrate": {"price": 60, "type": "list"},
+    "lurk": {"price": 1, "type": "counter", "info": "Let us know you're going to lurk."},
+    "react": {"price": 200, "type": "note", "info": "Attach link to a video for me to react to."},
+    "request": {"price": 100, "type": "note", "info": "Request a level, gamemode, skin, etc."},
+    "inactive": {"price": 100, "type": "note", "info": "Example redeem that is inactive by default", "category": ["inactive"]}
+}
+```
+#### File format
+`redeems.py` is a config file with just a `REDEEMS` key, that assigns a dictionary of redeems to it.
+Each dictionary entry is a redeem, and the dictionary keys are strings that decides the chat command for the redeem. The value is another dictionary that needs to have entries for `"price"`, `"type"` and optionally `"info"` and `"category"`.
 
-By default, everyone receives 10 points every 600 seconds (10 minutes).
+- `"price"` value should be an integer that decides how many points the redeem will cost.
+- `"type"` value should be either `"list"`, `"counter"` or `"note"`. This decided the redeem's type, and whether it will show up as a counter at the top of the dashboard or as an entry in the "recent redeems" chart.
+- `"info"` value should be a string that describes what the command does. It's optional, but I recommend writing one for all `"list"` and `"note"` redeems (so that chatters know that they should write a note).
+- `"category"` is an optional list of strings, the categories the redeem is in.
+If a category from the list is in `ACTIVE_CATEGORIES` from `config.py`,
+then the redeem will be active. It will not be active if none of the categories
+are in `ACTIVE_CATEGORIES`. Redeems with no category are always active.

setup.py

@@ -2,7 +2,7 @@ from setuptools import find_packages, setup
 
 setup(
     name='tlapbot',
-    version='0.6.1',
+    version='1.1.0',
     packages=find_packages(),
     include_package_data=True,
     install_requires=[

tlapbot/__init__.py

@@ -1,13 +1,14 @@
 import os
+import logging
 from flask import Flask
 from apscheduler.schedulers.background import BackgroundScheduler
 from tlapbot.db import get_db
-from tlapbot.owncast_helpers import is_stream_live, give_points_to_chat
-
+from tlapbot.owncast_helpers import (is_stream_live, give_points_to_chat,
+        remove_inactive_redeems)
 
 def create_app(test_config=None):
     app = Flask(__name__, instance_relative_config=True)
-
+    
     # ensure the instance folder exists
     try:
         os.makedirs(app.instance_path)
@@ -23,6 +24,21 @@ def create_app(test_config=None):
     app.config.from_object('tlapbot.default_redeems')
     app.config.from_pyfile('config.py', silent=True)
     app.config.from_pyfile('redeems.py', silent=True)
+    app.config.from_mapping(
+        REDEEMS=remove_inactive_redeems(app.config['REDEEMS'],
+                                        app.config['ACTIVE_CATEGORIES'])
+    )
+
+    # Make logging work for gunicorn-ran instances of tlapbot.
+    if app.config['GUNICORN']:
+        gunicorn_logger = logging.getLogger('gunicorn.error')
+        app.logger.handlers = gunicorn_logger.handlers
+        app.logger.setLevel(gunicorn_logger.level)
+    
+    # Check for wrong config that would break Tlapbot
+    if len(app.config['PREFIX']) != 1:
+        raise RuntimeError("Prefix is >1 character. "
+                           "Change your config to set 1-character prefix.")
 
     # prepare webhooks and redeem dashboard blueprints
     from . import owncast_webhooks
@@ -35,12 +51,16 @@ def create_app(test_config=None):
     db.init_app(app)
     app.cli.add_command(db.clear_queue_command)
     app.cli.add_command(db.refresh_counters_command)
+    app.cli.add_command(db.refresh_and_clear_command)
     
     # scheduler job for giving points to users
     def proxy_job():
         with app.app_context():
             if is_stream_live():
+                app.logger.info("Stream is LIVE. Giving points to chat.")
                 give_points_to_chat(get_db())
+            else:
+                app.logger.info("Stream is NOT LIVE. (Not giving points to chat.)")
 
     # start scheduler that will give points to users
     points_giver = BackgroundScheduler()

tlapbot/db.py

@@ -68,7 +68,7 @@ def refresh_counters():
         db.commit()
     except Error as e:
         print("Error occured deleting old counters:", e.args[0])
-    
+
     for redeem, redeem_info in current_app.config['REDEEMS'].items():
         if redeem_info["type"] == "counter":
             try:
@@ -100,11 +100,21 @@ def clear_queue_command():
 @click.command('refresh-counters')
 @with_appcontext
 def refresh_counters_command():
-    """Refresh counters from current config file. (Remove old ones, add new ones.)"""
+    """Refresh counters from current config file.
+    (Remove old ones, add new ones.)"""
     refresh_counters()
     click.echo('Counters refreshed.')
 
 
+@click.command('clear-refresh')
+@with_appcontext
+def refresh_and_clear_command():
+    """Refresh counters and clear queue."""
+    refresh_counters()
+    clear_redeem_queue()
+    click.echo('Counters refreshed and queue cleared.')
+
+
 def init_app(app):
     app.teardown_appcontext(close_db)
     app.cli.add_command(init_db_command)

tlapbot/default_config.py

@@ -3,4 +3,7 @@ OWNCAST_ACCESS_TOKEN=''
 OWNCAST_INSTANCE_URL='http://localhost:8080'
 POINTS_CYCLE_TIME=600
 POINTS_AMOUNT_GIVEN=10
-LIST_REDEEMS=False
+LIST_REDEEMS=False
+ACTIVE_CATEGORIES=[]
+GUNICORN=False
+PREFIX='!'

tlapbot/default_redeems.py

@@ -2,5 +2,6 @@ REDEEMS={
     "hydrate": {"price": 60, "type": "list"},
     "lurk": {"price": 1, "type": "counter", "info": "Let us know you're going to lurk."},
     "react": {"price": 200, "type": "note", "info": "Attach link to a video for me to react to."},
-    "request": {"price": 100, "type": "note", "info": "Request a level, gamemode, skin, etc."}
+    "request": {"price": 100, "type": "note", "info": "Request a level, gamemode, skin, etc."},
+    "inactive": {"price": 100, "type": "note", "info": "Example redeem that is inactive by default", "category": ["inactive"]}
 }

tlapbot/help_message.py

@@ -20,4 +20,4 @@ def send_help():
                 message.append(f"!{redeem} for {redeem_info['price']} points.\n")
     else:
         message.append("Check the dashboard for a list of currently active redeems.")
-    send_chat(''.join(message))
+    send_chat(''.join(message))

tlapbot/owncast_helpers.py

@@ -1,23 +1,20 @@
 from flask import current_app
 import requests
 from sqlite3 import Error
-from flask.cli import with_appcontext
-from tlapbot.db import get_db
-
+from re import sub
 
 # # # requests stuff # # #
 def is_stream_live():
     url = current_app.config['OWNCAST_INSTANCE_URL'] + '/api/status'
-    r = requests.post(url)
-    print(r.json()["online"])
+    r = requests.get(url)
     return r.json()["online"]
 
 
 def give_points_to_chat(db):
     url = current_app.config['OWNCAST_INSTANCE_URL'] + '/api/integrations/clients'
     headers = {"Authorization": "Bearer " + current_app.config['OWNCAST_ACCESS_TOKEN']}
-    r = requests.post(url, headers=headers)
-    unique_users = list(set(map(lambda user_object: user_object["user"]["id"], r.json())))
+    r = requests.get(url, headers=headers)
+    unique_users = set(map(lambda user_object: user_object["user"]["id"], r.json()))
     for user_id in unique_users:
         give_points_to_user(db,
                             user_id,
@@ -41,8 +38,8 @@ def read_users_points(db, user_id):
         )
         return cursor.fetchone()[0]
     except Error as e:
-        print("Error occured reading points:", e.args[0])
-        print("To user:", user_id)
+        current_app.logger.error(f"Error occured reading points: {e.args[0]}")
+        current_app.logger.error(f"To user: {user_id}")
 
 
 def read_all_users_with_username(db, username):
@@ -54,8 +51,9 @@ def read_all_users_with_username(db, username):
         users = cursor.fetchall()
         return users
     except Error as e:
-        print("Error occured reading points from username:", e.args[0])
-        print("To user:", username)
+        current_app.logger.error(f"Error occured reading points by username: {e.args[0]}")
+        current_app.logger.error(f"To everyone with username: {username}")
+
 
 def give_points_to_user(db, user_id, points):
     try:
@@ -65,8 +63,8 @@ def give_points_to_user(db, user_id, points):
         )
         db.commit()
     except Error as e:
-        print("Error occured giving points:", e.args[0])
-        print("To user:", user_id, "  amount of points:", points)
+        current_app.logger.error(f"Error occured giving points: {e.args[0]}")
+        current_app.logger.error(f"To user: {user_id} amount of points: {points}")
 
 
 def use_points(db, user_id, points):
@@ -78,8 +76,8 @@ def use_points(db, user_id, points):
         db.commit()
         return True
     except Error as e:
-        print("Error occured using points:", e.args[0])
-        print("From user:", user_id, "  amount of points:", points)
+        current_app.logger.error(f"Error occured using points: {e.args[0]}")
+        current_app.logger.error(f"From user: {user_id} amount of points: {points}")
         return False
 
 
@@ -93,8 +91,8 @@ def user_exists(db, user_id):
             return False
         return True
     except Error as e:
-        print("Error occured checking if user exists:", e.args[0])
-        print("To user:", user_id)
+        current_app.logger.error(f"Error occured checking if user exists: {e.args[0]}")
+        current_app.logger.error(f"To user: {user_id}")
 
 
 def add_user_to_database(db, user_id, display_name):
@@ -110,7 +108,7 @@ def add_user_to_database(db, user_id, display_name):
                 "INSERT INTO points(id, name, points) VALUES(?, ?, 10)",
                 (user_id, display_name)
             )
-        if user is not None and user[1] == None:
+        if user is not None and user[1] is None:
             cursor.execute(
                 """UPDATE points
                 SET name = ?
@@ -119,8 +117,8 @@ def add_user_to_database(db, user_id, display_name):
             )
         db.commit()
     except Error as e:
-        print("Error occured adding user to db:", e.args[0])
-        print("To user:", user_id, display_name)
+        current_app.logger.error(f"Error occured adding user to db: {e.args[0]}")
+        current_app.logger.error(f"To user id: {user_id}, with display name: {display_name}")
 
 
 def change_display_name(db, user_id, new_name):
@@ -131,20 +129,41 @@ def change_display_name(db, user_id, new_name):
         )
         db.commit()
     except Error as e:
-        print("Error occured changing display name:", e.args[0])
-        print("To user:", user_id, new_name)
+        current_app.logger.error(f"Error occured changing display name: {e.args[0]}")
+        current_app.logger.error(f"To user id: {user_id}, with display name: {new_name}")
+
+
+def check_counter_exists(db, counter_name):
+    try:
+        cursor = db.execute(
+            "SELECT count FROM counters WHERE name = ?",
+            (counter_name,)
+        )
+        counter = cursor.fetchone()
+        if counter is None:
+            current_app.logger.warning("Counter not found in database.")
+            current_app.logger.warning("Maybe you forgot to run the refresh-counters CLI command "
+                                       "after you added a new counter to the config?")
+            return False
+        return True
+    except Error as e:
+        current_app.logger.error(f"Error occured checking if counter exists: {e.args[0]}")
+        current_app.logger.error(f"For counter: {counter_name}")
 
 
 def add_to_counter(db, counter_name):
-    try:
-        cursor = db.execute(
-            "UPDATE counters SET count = count + 1 WHERE name = ?",
-            (counter_name,)
-        )
-        db.commit()
-    except Error as e:
-        print("Error occured adding to counter:", e.args[0])
-        print("To counter:", counter_name)
+    if check_counter_exists(db, counter_name):
+        try:
+            cursor = db.execute(
+                "UPDATE counters SET count = count + 1 WHERE name = ?",
+                (counter_name,)
+            )
+            db.commit()
+            return True
+        except Error as e:
+            current_app.logger.error(f"Error occured adding to counter: {e.args[0]}")
+            current_app.logger.error(f"To counter: {counter_name}")
+    return False
 
 
 def add_to_redeem_queue(db, user_id, redeem_name, note=None):
@@ -154,11 +173,11 @@ def add_to_redeem_queue(db, user_id, redeem_name, note=None):
             (redeem_name, user_id, note)
             )
         db.commit()
+        return True
     except Error as e:
-        print("Error occured adding to redeem queue:", e.args[0])
-        print("To user:", user_id, " with redeem:", redeem_name, "with note:", note)
-
-
+        current_app.logger.error(f"Error occured adding to redeem queue: {e.args[0]}")
+        current_app.logger.error(f"To user: {user_id} with redeem: {redeem_name} with note: {note}")
+        return False
 
 
 def all_counters(db):
@@ -168,7 +187,7 @@ def all_counters(db):
         )
         return cursor.fetchall()
     except Error as e:
-        print("Error occured selecting all counters:", e.args[0])
+        current_app.logger.error(f"Error occured selecting all counters: {e.args[0]}")
 
 
 def pretty_redeem_queue(db):
@@ -181,7 +200,7 @@ def pretty_redeem_queue(db):
         )
         return cursor.fetchall()
     except Error as e:
-        print("Error occured selecting pretty redeem queue:", e.args[0])
+        current_app.logger.error(f"Error occured selecting pretty redeem queue: {e.args[0]}")
 
 
 def whole_redeem_queue(db):
@@ -191,7 +210,7 @@ def whole_redeem_queue(db):
         )
         return cursor.fetchall()
     except Error as e:
-        print("Error occured selecting redeem queue:", e.args[0])
+        current_app.logger.error(f"Error occured selecting redeem queue: {e.args[0]}")
 
 
 def remove_duplicate_usernames(db, user_id, username):
@@ -200,8 +219,31 @@ def remove_duplicate_usernames(db, user_id, username):
             """UPDATE points
             SET name = NULL
             WHERE name = ? AND NOT id = ?""",
-        (username, user_id)
+            (username, user_id)
         )
         db.commit()
     except Error as e:
-        print("Error occured removing duplicate usernames:", e.args[0])
+        current_app.logger.error(f"Error occured removing duplicate usernames: {e.args[0]}")
+
+
+# # # misc. stuff # # #
+def remove_emoji(message):
+    return sub(
+        r'<img class="emoji" alt="(:.*?:)" title=":.*?:" src="/img/emoji/.*?">',
+        r'\1',
+        message
+    )
+
+
+def is_redeem_active(redeem, active_categories):
+    if "category" in redeem[1] and redeem[1]["category"]:
+        for category in redeem[1]["category"]:
+            if category in active_categories:
+                return True
+        return False
+    return True
+
+
+def remove_inactive_redeems(redeems, active_categories):
+    return dict(filter(lambda redeem: is_redeem_active(redeem, active_categories),
+                       redeems.items()))

tlapbot/owncast_redeem_dashboard.py

@@ -10,19 +10,17 @@ bp = Blueprint('redeem_dashboard', __name__)
 @bp.route('/dashboard', methods=['GET'])
 def dashboard():
     db = get_db()
-    queue = pretty_redeem_queue(db)
-    counters = all_counters(db)
-    redeems = current_app.config['REDEEMS']
     username = request.args.get("username")
     if username is not None:
         users = read_all_users_with_username(db, username)
-    else: 
+    else:
         users = []
     utc_timezone = timezone.utc
     return render_template('dashboard.html',
-                           queue=queue,
-                           counters=counters,
-                           redeems=redeems,
+                           queue=pretty_redeem_queue(db),
+                           counters=all_counters(db),
+                           redeems=current_app.config['REDEEMS'],
+                           prefix=current_app.config['PREFIX'],
                            username=username,
                            users=users,
                            utc_timezone=utc_timezone)

tlapbot/owncast_webhooks.py

@@ -1,8 +1,7 @@
 from flask import Flask, request, json, Blueprint, current_app
-from sqlite3 import Error
 from tlapbot.db import get_db
 from tlapbot.owncast_helpers import (add_user_to_database, change_display_name,
-        user_exists, send_chat, read_users_points, remove_duplicate_usernames)
+        send_chat, read_users_points, remove_duplicate_usernames)
 from tlapbot.help_message import send_help
 from tlapbot.redeems_handler import handle_redeem
 
@@ -14,11 +13,14 @@ bp = Blueprint('owncast_webhooks', __name__)
 def owncast_webhook():
     data = request.json
     db = get_db()
-    if data["type"] == "USER_JOINED":
+
+    # Make sure user is in db before doing anything else.
+    if data["type"] in ["CHAT", "NAME_CHANGED", "USER_JOINED"]:
         user_id = data["eventData"]["user"]["id"]
         display_name = data["eventData"]["user"]["displayName"]
-        # CONSIDER: join points for joining stream
         add_user_to_database(db, user_id, display_name)
+
+    if data["type"] == "USER_JOINED":
         if data["eventData"]["user"]["authenticated"]:
             remove_duplicate_usernames(db, user_id, display_name)
     elif data["type"] == "NAME_CHANGE":
@@ -28,25 +30,26 @@ def owncast_webhook():
         if data["eventData"]["user"]["authenticated"]:
             remove_duplicate_usernames(db, user_id, new_name)
     elif data["type"] == "CHAT":
+        prefix = current_app.config['PREFIX']
         user_id = data["eventData"]["user"]["id"]
         display_name = data["eventData"]["user"]["displayName"]
-        print(f'New chat message from {display_name}:')
-        print(f'{data["eventData"]["body"]}')
-        if "!help" in data["eventData"]["body"]:
+        current_app.logger.debug(f'New chat message from {display_name}:')
+        current_app.logger.debug(f'{data["eventData"]["body"]}')
+        if data["eventData"]["body"].startswith(f"{prefix}help"):
             send_help()
-        elif "!points" in data["eventData"]["body"]:
-            if not user_exists(db, user_id):
-                add_user_to_database(db, user_id, display_name)
+        elif data["eventData"]["body"].startswith(f"{prefix}points"):
             points = read_users_points(db, user_id)
-            message = f"{display_name}'s points: {points}"
-            send_chat(message)
-        elif "!name_update" in data["eventData"]["body"]:
+            if points is None:
+                send_chat("Error reading points.")
+            else:
+                send_chat(f"{display_name}'s points: {points}")
+        elif data["eventData"]["body"].startswith(f"{prefix}name_update"):
             # Forces name update in case bot didn't catch the NAME_CHANGE
             # event. Also removes saved usernames from users with same name
             # if user is authenticated.
             change_display_name(db, user_id, display_name)
             if data["eventData"]["user"]["authenticated"]:
                 remove_duplicate_usernames(db, user_id, display_name)
-        elif data["eventData"]["body"].startswith("!"):  # TODO: make prefix configurable
+        elif data["eventData"]["body"].startswith(prefix):
             handle_redeem(data["eventData"]["body"], user_id)
     return data

tlapbot/redeems_handler.py

@@ -1,7 +1,7 @@
 from flask import current_app
 from tlapbot.db import get_db
 from tlapbot.owncast_helpers import (use_points, add_to_redeem_queue,
-        add_to_counter, read_users_points, send_chat)
+        add_to_counter, read_users_points, send_chat, remove_emoji)
 
 
 def handle_redeem(message, user_id):
@@ -12,30 +12,38 @@ def handle_redeem(message, user_id):
     else:
         note = split_message[1]
 
-    if redeem in current_app.config['REDEEMS']:
-        db = get_db()
-        price = current_app.config['REDEEMS'][redeem]["price"]
-        redeem_type = current_app.config['REDEEMS'][redeem]["type"]
-        points = read_users_points(db, user_id)
-        if points is not None and points >= price:
-            if redeem_type == "counter":
-                add_to_counter(db, redeem)
-                use_points(db, user_id, price)
-                send_chat(f"{redeem} redeemed for {price} points.")
-            elif redeem_type == "list":
-                add_to_redeem_queue(db, user_id, redeem)
-                use_points(db, user_id, price)
-                send_chat(f"{redeem} redeemed for {price} points.")
-            elif redeem_type == "note":
-                if note is not None:
-                    add_to_redeem_queue(db, user_id, redeem, note)
-                    use_points(db, user_id, price)
-                    send_chat(f"{redeem} redeemed for {price} points.")
-                else:
-                    send_chat(f"Cannot redeem {redeem}, no note included.")
-            else:
-                send_chat(f"{redeem} not redeemed because of an error.")
-        else:
-            send_chat(f"Can't redeem {redeem}, you don't have enough points.")
-    else:
+    if redeem not in current_app.config['REDEEMS']:
         send_chat("Can't redeem, redeem not found.")
+        return
+
+    db = get_db()
+    price = current_app.config['REDEEMS'][redeem]["price"]
+    redeem_type = current_app.config['REDEEMS'][redeem]["type"]
+    points = read_users_points(db, user_id)
+
+    if not points or points < price:
+        send_chat(f"Can't redeem {redeem}, you don't have enough points.")
+        return
+
+    if redeem_type == "counter":
+        if add_to_counter(db, redeem) and use_points(db, user_id, price):
+            send_chat(f"{redeem} redeemed for {price} points.")
+        else:
+            send_chat(f"Redeeming {redeem} failed.")
+    elif redeem_type == "list":
+        if (add_to_redeem_queue(db, user_id, redeem) and
+                use_points(db, user_id, price)):
+            send_chat(f"{redeem} redeemed for {price} points.")
+        else:
+            send_chat(f"Redeeming {redeem} failed.")
+    elif redeem_type == "note":
+        if not note:
+            send_chat(f"Cannot redeem {redeem}, no note included.")
+            return
+        if (add_to_redeem_queue(db, user_id, redeem, remove_emoji(note)) and
+                use_points(db, user_id, price)):
+            send_chat(f"{redeem} redeemed for {price} points.")
+        else:
+            send_chat(f"Redeeming {redeem} failed.")
+    else:
+        send_chat(f"{redeem} not redeemed, type of redeem not found.")

tlapbot/static/dashboard.js

@@ -15,5 +15,9 @@ function openTab(event, tabName) {
   
     // Show the current tab, and add an "active" class to the button that opened the tab
     document.getElementById(tabName).style.display = "block";
-    evt.currentTarget.className += " active";
-  }
+    event.currentTarget.className += " active";
+  }
+
+function refreshPage() {
+    window.location.reload();
+}

tlapbot/templates/dashboard.html

@@ -97,7 +97,7 @@
                 </thead>
                 {% for redeem, redeem_info in redeems.items() %}
                 <tbody>
-                    <td>!{{ redeem }}</td>
+                    <td>{{ prefix }}{{ redeem }}</td>
                     <td>{{ redeem_info["price"] }}</td>
                     <td>{{ redeem_info["type"] }}</td>
                     {% if redeem_info["info"] %}
@@ -110,10 +110,11 @@
         </body>
     </div>
 
+    <script>
+        document.getElementById("defaultOpen").click();
+        setTimeout(refreshPage, 30 * 1000);
+    </script>
+
 </div>
 
-<script>
-    document.getElementById("defaultOpen").click();
-</script>
-
 </html>