Add callout id functionality to the bot, enabling users to specify an ID for their callouts during creation/management & remove "potential fill" functionality #2

Merged

contrastellar merged 31 commits from gabby-dev into main 2026-05-29 17:48:00 +00:00

Conversation 0 Commits 31 Files changed 14 +51 -1609

contrastellar commented 2026-05-29 17:10:44 +00:00

Owner

Copy link

This enables users to specify an ID during removal of callouts, which will make removal easier. This is largely because the date fields are slightly incorrect.
This also removes the 'potential fill' functionality from the bot, but does not remove those columns from the production database.

This enables users to specify an ID during removal of callouts, which will make removal easier. This is largely because the date fields are slightly incorrect. This also removes the 'potential fill' functionality from the bot, but does not remove those columns from the production database.

contrastellar added the

Kind/Feature

label 2026-05-29 17:10:44 +00:00

contrastellar self-assigned this 2026-05-29 17:10:44 +00:00

contrastellar added 26 commits 2026-05-29 17:10:44 +00:00

Update makefile to use poetry commands bd59046cc3

Rename from github to forgejo to follow conventions 4da235b6f6

Remove poetry.lock (which would need to be generated on new runtime environments anyways), update .gitignore e5a21216fd

Update pyproject.toml c7201ae45c

Update deploy.yaml; remove old conda call, update poetry commands to update after install 3064b266eb

Update test.yaml; poetry will update after install 92f28c8b77

Remove the cleanup command, as it would break how serialization works in the future 49608c7606

Remove requirements. Not necessary for poetry 99fade3cea

Remove breaks function, as it is unused eb3f702ced

Remove break functionality from bot core 4ce8cfb8cd

Update message text for remote_registration 3615042ae6

remove tests for deprecated methods a4fbbd13fd

remove old methods from the db_helper module 7aff8a33ed

Add todo messages for the bot core 056960ae55

fix minor typo 167a578439

Remove SQL scripts, no longer need to be tracked 0c5725656b

fix: undo removal of add_callout method 2ad10eba2e

Remove potential fill functionality from the /callout command 0565be6938

Update db_helper to return the ID instead of the fill, as fill has been deprecated 91e6873a65

Add functionality to support returning the callout ID during a callout creation 1f9a84f18e

Remove the schedule followup during a callout creation, unncessecary invokation 11e7ab3d64

Edit comments in bot_core 742b07865b

Adjust add_callout method in db_helper to accomodate the lack of fills going forward e675f71de2

Update remove_callout() -> remove_callout_by_date e962afa816

Update method signature and references for remove_callout -> remove_callout_by_date inside the db_helper method da8ec5bcb6

Create remove_callout() command for bot and remove_callout_by_id() method in the db_helper 3c7b26ce37

contrastellar added 1 commit 2026-05-29 17:12:15 +00:00

Fix the test for the removal of callouts 6869fa7f86

contrastellar added 1 commit 2026-05-29 17:14:26 +00:00

remove extraneous parameter passing for callout_add test b32fccfabb

contrastellar added 3 commits 2026-05-29 17:40:21 +00:00

update asset to reflect correct return type of the add_callout method 3bc2081c35

Update assert statements to actually test types, not object equivalence d7cc786d27

Force add_callout() to return a hard-casted int 7c967585ca

contrastellar changed title from WIP: Add callout id functionality to the bot, enabling users to specify an ID for their callouts during creation/management & remove "potential fill" functionality to Add callout id functionality to the bot, enabling users to specify an ID for their callouts during creation/management & remove "potential fill" functionality 2026-05-29 17:42:02 +00:00

contrastellar merged commit 2b672ef4f3 into main 2026-05-29 17:48:00 +00:00

contrastellar referenced this pull request from a commit 2026-05-29 17:48:01 +00:00

Merge pull request 'Add `callout id` functionality to the bot, enabling users to specify an ID for their callouts during creation/management & remove "potential fill" functionality' (#2) from gabby-dev into main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

Compat/Breaking

Breaking change that won't be backward compatible

  

Kind/Bug

Something is not working

  

Kind/Documentation

Documentation changes

  

Kind/Enhancement

Improve existing functionality

  

Kind/Feature

New functionality

  

Kind/Security

This is security issue

  

Kind/Testing

Issue or pull request related to testing

  

Priority

Critical

The priority is critical

  

Priority

High

The priority is high

  

Priority

Low

The priority is low

  

Priority

Medium

The priority is medium

  

Reviewed

Confirmed

Issue has been confirmed

  

Reviewed

Duplicate

This issue or pull request already exists

  

Reviewed

Invalid

Invalid issue

  

Reviewed

Won't Fix

This issue won't be fixed

  

Status

Abandoned

Somebody has started to work on this but abandoned work

  

Status

Blocked

Something is blocking this issue or pull request

  

Status

Need More Info

Feedback is required to reproduce issue or to continue work

No labels

Compat/Breaking

Kind/Bug

Kind/Documentation

Kind/Enhancement

Kind/Feature

Kind/Security

Kind/Testing

Priority

Critical

Priority

High

Priority

Low

Priority

Medium

Reviewed

Confirmed

Reviewed

Duplicate

Reviewed

Invalid

Reviewed

Won't Fix

Status

Abandoned

Status

Blocked

Status

Need More Info

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

contrastellar

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

contrastellar/raid-callouts!2

No description provided.