Conversation
There was a problem hiding this comment.
Pull request overview
Adds an admin-managed “Settings” area in the Slack App Home to configure stale ticket auto-close, persisting configuration in the database and making the stale-close task use the configured threshold.
Changes:
- Introduces a new Prisma
Settingsmodel for key/value configuration storage. - Adds Slack App Home “Settings” tab + modal/actions to configure/enable/disable
stale_ticket_days. - Enables and updates the stale-ticket closer task to use the DB-configured threshold and run on a scheduler.
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
prisma/schema.prisma |
Adds DB model for persisted settings (key/value + timestamps). |
nephthys/views/modals/configure_stale_days.py |
New modal UI for editing stale-days threshold. |
nephthys/views/home/settings.py |
New App Home Settings view (admin-gated) showing stale-close status and actions. |
nephthys/views/home/__init__.py |
Adds “Settings” to App Home navigation. |
nephthys/utils/slack.py |
Registers new Slack action + view handlers for settings flows. |
nephthys/utils/env.py |
Adds helper to read/parse stale_ticket_days from DB settings. |
nephthys/tasks/close_stale.py |
Uses configured stale-days threshold; disables task when unset. |
nephthys/events/app_home_opened.py |
Routes App Home navigation to the new settings view. |
nephthys/actions/settings.py |
Implements Slack callbacks to configure / toggle the stale-close feature. |
nephthys/__main__.py |
Schedules the stale-close job to run hourly. |
docs/deployment.md |
Documents how to configure bot settings from Slack App Home. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 11 out of 11 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
MMK21Hub
left a comment
There was a problem hiding this comment.
Just forwarding the context from DMs:
Heads up that #175 (stale tickets config) introduces a new config system to the DB which I don't want to do rn (I do want to add a new config system at some point, but that requires a lot of thought)
right now similar things are configured through environment variables so I don't want to add a whole new mechanism just to configure this
happy to accept this w/o the new config system and just with an env var to configure it!
No description provided.