Skip to content

[WIP] Update to address feedback on Edit return requirement for Pin Message #240

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
Copilot wants to merge 1 commit into from
Closed

[WIP] Update to address feedback on Edit return requirement for Pin Message #240

Copilot wants to merge 1 commit into from
+0 −0

Conversation

Copy link

Copilot AI commented Feb 2, 2026 •
edited
Loading

  • Understand the "Edit" requirement for modal responses in pin feature
  • Check PR [v9] スケジュール投稿機能の追加 #225 (schedule) for the correct pattern
  • Update pin_modal.go to use deferred response + EditInteractionResponse
  • Update pin_select.go to use deferred response + EditInteractionResponse
  • Resolve remaining @copilot review comments
    • Bot message check in messageCreate.go
    • Unpin command should delete Discord messages
    • Unused replyPinSuccess function
    • Modal placeholder text inconsistency
    • DeleteByChannelID test coverage
  • Test the changes
  • Run code review
  • Run CodeQL security scan

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI mentioned this pull request Feb 2, 2026
Open
@yuito-it yuito-it closed this Feb 2, 2026
Copilot AI requested a review from yuito-it February 2, 2026 11:48
Copilot stopped work on behalf of yuito-it due to an error February 2, 2026 11:48
The session was cancelled by the user.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yuito-it yuito-it Awaiting requested review from yuito-it

Assignees

@yuito-it yuito-it

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@yuito-it