Skip to content

Update index.ex#2435

Closed
sydseter wants to merge 3 commits intomasterfrom
remove-delete-game
Closed

Update index.ex#2435
sydseter wants to merge 3 commits intomasterfrom
remove-delete-game

Conversation

@sydseter
Copy link
Collaborator

No description provided.

@sydseter sydseter requested a review from rewtd as a code owner February 27, 2026 21:56
Copilot AI review requested due to automatic review settings February 27, 2026 21:56
@sydseter sydseter requested a review from cw-owasp as a code owner February 27, 2026 21:56
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes the handle_event/3 handler for the "delete" LiveView event from CopiWeb.GameLive.Index, effectively disabling game deletion through this LiveView module.

Changes:

  • Deleted the "delete" handle_event/3 implementation that fetched and deleted a game by id.
  • Leaves list_games/0 in place (now potentially unused after the handler removal).

Copy link
Contributor

Copilot AI left a comment

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 1 out of 1 changed files in this pull request and generated no new comments.

@sydseter sydseter closed this Mar 1, 2026
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.

2 participants