RSS Remove Command #532

New issue

Closed

opened 2025-11-24 21:56:18 +00:00 by Vylpes · 2 comments

Vylpes commented 2025-11-24 21:56:18 +00:00

Owner

Copy link

Epic: #528
Story Points:

---

Create a slash command for moderators to unsubscribe from RSS feeds

Acceptance Criteria

GIVEN I am a moderator
WHEN I run /rss remove id:<ID>
THEN the feed <ID> will be removed from the database

GIVEN I am a moderator
WHEN I run /rss remove id:<ID>
AND the <ID> feed doesn't exist
THEN an error will occur

Subtasks

• Create /rss remove command in src/commands/
• Implement moderator permission check
• Delete feed subscription from database
• Send confirmation message
• Add error handling for non-existent feeds
• Update tests

Notes

• The parameters for /rss remove are: id (string, required)
• feedId will be saved as the same format as in audits

Epic: #528 Story Points: --- Create a slash command for moderators to unsubscribe from RSS feeds ## Acceptance Criteria GIVEN I am a moderator WHEN I run `/rss remove id:<ID>` THEN the feed `<ID>` will be removed from the database GIVEN I am a moderator WHEN I run `/rss remove id:<ID>` AND the `<ID>` feed doesn't exist THEN an error will occur ## Subtasks - [x] Create `/rss remove` command in `src/commands/` - [x] Implement moderator permission check - [x] Delete feed subscription from database - [x] Send confirmation message - [x] Add error handling for non-existent feeds - [x] Update tests ## Notes - The parameters for `/rss remove` are: `id` (string, required) - `feedId` will be saved as the same format as in audits

Vylpes added a new dependency 2025-11-24 21:56:44 +00:00

#531 RSS Add Command

Vylpes added a new dependency 2025-11-24 22:15:50 +00:00

#536 RSS Testing & Documentation

Vylpes referenced this issue 2025-11-24 22:17:02 +00:00

RSS Feeds #528

Vylpes added this to the 3.4.0 milestone 2025-11-24 22:17:40 +00:00

Vylpes added the

blocked

type

story

labels 2026-09-17 18:25:30 +01:00

Vylpes added

step

todo

and removed

blocked

labels 2026-10-25 13:03:12 +00:00

Smithy-bot was assigned by Vylpes 2026-10-25 13:03:18 +00:00

Smithy-bot added

step

doing

and removed

step

todo

labels 2026-10-30 11:04:48 +00:00

Smithy-bot started working 2026-10-30 11:04:49 +00:00

Smithy-bot referenced this issue from a commit 2026-10-30 11:06:22 +00:00

Add /rss remove command (#532)

Smithy-bot referenced this issue 2026-10-30 11:06:27 +00:00

Add /rss remove command (#532) #594

Smithy-bot commented 2026-10-30 11:06:39 +00:00

Member

Copy link

Implemented /rss remove on branch cursor/532-rss-remove-command (from feature/epic-528-rss-feeds, same base as the other RSS stories).

PR: #594

• Adds remove subcommand with required id
• Looks up FeedId case-insensitively within the guild; replies RSS feed not found. when missing
• Deletes the subscription and confirms with an embed (ID / URL / Channel)
• Moderator-only via existing ModerateMembers permission on /rss
• Local yarn build + yarn test PASS (9 suites / 107 tests)

Moved to step/review and assigned to @Vylpes. Please review/approve — I have not approved or merged.

Implemented `/rss remove` on branch `cursor/532-rss-remove-command` (from `feature/epic-528-rss-feeds`, same base as the other RSS stories). PR: https://git.vylpes.xyz/RabbitLabs/vylbot-app/pulls/594 - Adds `remove` subcommand with required `id` - Looks up `FeedId` case-insensitively within the guild; replies `RSS feed not found.` when missing - Deletes the subscription and confirms with an embed (ID / URL / Channel) - Moderator-only via existing `ModerateMembers` permission on `/rss` - Local `yarn build` + `yarn test` PASS (9 suites / 107 tests) Moved to `step/review` and assigned to @Vylpes. Please review/approve — I have not approved or merged.

Smithy-bot added

step

review

and removed

step

doing

labels 2026-10-30 11:06:39 +00:00

Smithy-bot removed their assignment 2026-10-30 11:06:39 +00:00

Vylpes was assigned by Smithy-bot 2026-10-30 11:06:39 +00:00

Smithy-bot stopped working 2026-10-30 11:06:39 +00:00

1 minute 50 seconds

Vylpes referenced this issue from a commit 2026-10-30 13:56:04 +00:00

Merge pull request 'Add /rss remove command (#532)' (#594) from cursor/532-rss-remove-command into feature/epic-528-rss-feeds

Vylpes added

step

testing

and removed

step

review

labels 2026-10-30 13:56:11 +00:00

Vylpes removed their assignment 2026-10-30 13:56:15 +00:00

Smithy-bot was assigned by Vylpes 2026-10-30 13:56:15 +00:00

Smithy-bot started working 2026-10-31 11:02:16 +00:00

Smithy-bot stopped working 2026-10-31 11:02:46 +00:00

30 seconds

Smithy-bot added

step

uat

and removed

step

testing

labels 2026-10-31 11:02:51 +00:00

Smithy-bot commented 2026-10-31 11:02:51 +00:00

Member

Copy link

QA results — PASS

Tested on feature/epic-528-rss-feeds @ c3de1d4 (PR #594 merge).

Automated

• yarn install --frozen-lockfile PASS
• yarn build (tsc) PASS
• yarn test PASS — 9 suites / 107 tests / 2 snapshots
• tests/commands/rss.test.ts remove cases: 6/6 PASS (missing id, not found, remove fail, successful remove + confirmation embed)
• No yarn lint script in this repo

Acceptance criteria

• /rss remove subcommand with required id is registered; /rss gated by ModerateMembers
• Existing feed id → deleted via RssFeed.Remove and confirmation embed (ID / URL / Channel)
• Missing feed id → RSS feed not found. (case-insensitive lookup)

Moved step/testing → step/uat. Live Discord /rss remove smoke left for human UAT.

## QA results — PASS Tested on `feature/epic-528-rss-feeds` @ `c3de1d4` (PR #594 merge). ### Automated - `yarn install --frozen-lockfile` PASS - `yarn build` (tsc) PASS - `yarn test` PASS — 9 suites / 107 tests / 2 snapshots - `tests/commands/rss.test.ts` remove cases: 6/6 PASS (missing id, not found, remove fail, successful remove + confirmation embed) - No `yarn lint` script in this repo ### Acceptance criteria - `/rss remove` subcommand with required `id` is registered; `/rss` gated by `ModerateMembers` - Existing feed id → deleted via `RssFeed.Remove` and confirmation embed (ID / URL / Channel) - Missing feed id → `RSS feed not found.` (case-insensitive lookup) Moved `step/testing` → `step/uat`. Live Discord `/rss remove` smoke left for human UAT.

Smithy-bot removed their assignment 2026-10-31 11:02:51 +00:00

Vylpes was assigned by Smithy-bot 2026-10-31 11:02:51 +00:00

Smithy-bot added spent time 2026-10-31 11:03:12 +00:00

1 minute 30 seconds

Vylpes 2026-10-31 11:55:47 +00:00

• closed this issue
• removed the
  step
  uat
  label

Sign in to join this conversation.

No Branch/Tag specified

Branches Tags

main

feature/epic-528-rss-feeds

release/3.4.0

release/3.5.0

develop

v3.3.1

v3.3.0

v3.2.5

v3.2.4

v3.2.3

v3.2.2

v3.2.1

v3.2.0

v3.1.3

v3.1.2

v3.1.1

v3.1.0

v3.0.7

v3.0.6

v3.0.5

v3.1-alpha1

v3.0.4

v3.0.3

v3.0.2

v3.0.1

v3.0

v3.0-rc2

v3.0-rc1

v3.0-beta2

v3.0-beta1

v2.1.3

Labels

Clear labels

  

blocked

  

duplicate

  

high priority

  

needs

approval

  

needs

criteria

  

needs

estimate

  

needs

tests

  

question

  

step

doing

  

step

review

  

step

testing

  

step

todo

  

step

uat

  

type

admin

  

type

alert

  

type

bug

  

type

change

  

type

defect

  

type

dependencies

  

type

development

  

type

documentation

  

type

epic

  

type

idea

  

type

incident

  

type

investigation

  

type

spike

  

type

story

  

type

tests

  

won't fix

No labels

blocked

duplicate

high priority

needs

approval

needs

criteria

needs

estimate

needs

tests

question

step

doing

step

review

step

testing

step

todo

step

uat

type

admin

type

alert

type

bug

type

change

type

defect

type

dependencies

type

development

type

documentation

type

epic

type

idea

type

incident

type

investigation

type

spike

type

story

type

tests

won't fix

Milestone

Clear milestone

No items

No milestone

3.4.0

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

Vylpes

2 participants

Notifications

Subscribe

Total time spent: 3 minutes 50 seconds

Smithy-bot

3 minutes 50 seconds

Due date

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

No due date set.

Blocks

#536 RSS Testing & Documentation

RabbitLabs/vylbot-app

Depends on

#531 RSS Add Command

RabbitLabs/vylbot-app

Reference

RabbitLabs/vylbot-app#532

No description provided.