Skip to content

Ticket#2

Merged
robo38 merged 14 commits into
mainfrom
fenz-dev
Mar 26, 2026
Merged

Ticket#2
robo38 merged 14 commits into
mainfrom
fenz-dev

Conversation

@FenzDev

@FenzDev FenzDev commented Mar 26, 2026

Copy link
Copy Markdown
Member

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a new Discord “ticket” feature area (configs + /ticket create flow) and introduces open/close ticket component handlers, while also pinning the discord.js dependency version.

Changes:

  • Reformats TicketRepository and adds ticket config constants + ticket categories.
  • Adds /ticket create command and modal + “open ticket” / “close ticket” component handlers.
  • Pins discord.js to ^14.25.1 (but the lockfile update is currently broken due to merge conflicts).

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 17 comments.

Show a summary per file
File Description
src/database/repositories/TicketRepository.ts Reformatting of repository methods (no functional changes).
src/bot/ticket/config/misc.ts Adds ticket-related Discord IDs and UI constants.
src/bot/ticket/config/categories.ts Defines ticket categories used by the ticket UI.
src/bot/ticket/components/openTicket.ts Handles modal submit to create a ticket channel + DB record + staff report.
src/bot/ticket/components/closeTicket.ts Handles “Close” button to close ticket + delete channel + staff report.
src/bot/ticket/commands/ticket.ts Adds /ticket create command and constructs the modal UI.
package.json Pins discord.js version.
package-lock.json Lockfile changes, but currently contains unresolved merge conflicts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/bot/ticket/components/closeTicket.ts Outdated
Comment thread src/bot/ticket/components/openTicket.ts Outdated
Comment thread src/bot/ticket/commands/ticket.ts Outdated
Comment thread src/bot/ticket/components/closeTicket.ts
Comment thread src/bot/ticket/config/categories.ts
Comment thread src/bot/ticket/config/categories.ts Outdated
Comment thread src/bot/ticket/components/closeTicket.ts
Comment thread src/bot/ticket/components/openTicket.ts Outdated
Comment thread src/database/repositories/TicketRepository.ts Outdated
Comment thread src/bot/ticket/components/openTicket.ts Outdated
@robo38 robo38 requested review from Copilot and robo38 March 26, 2026 22:56
@robticorg robticorg deleted a comment from Copilot AI Mar 26, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 6 out of 8 changed files in this pull request and generated 7 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/bot/ticket/components/openTicket.ts
Comment thread src/bot/ticket/components/openTicket.ts
Comment thread src/bot/ticket/components/closeTicket.ts
Comment thread src/bot/ticket/commands/ticket.ts
Comment thread src/bot/ticket/commands/ticket.ts
Comment thread src/bot/ticket/components/openTicket.ts
Comment thread src/bot/ticket/components/openTicket.ts
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

@robo38 robo38 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

done

@robo38 robo38 merged commit 679d93c into main Mar 26, 2026
2 of 3 checks passed

@robo38 robo38 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants