Skip to content

Conversation

@Pugma
Copy link
Collaborator

@Pugma Pugma commented Sep 3, 2025

as titled

@Pugma Pugma requested a review from Copilot September 3, 2025 08:51
@Pugma Pugma changed the title deprecate(oapi): GET /words, /list/user/:userId & /list/word/word deprecate(oapi): GET /words, /list/user/:userId & /list/word/:word Sep 3, 2025

This comment was marked as outdated.

@Pugma Pugma requested a review from Copilot September 3, 2025 09:10

This comment was marked as outdated.

Copy link
Contributor

@YuHima03 YuHima03 left a comment

Choose a reason for hiding this comment

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

コメントしたので確認していただけると嬉しいです 🙇

@Pugma Pugma requested a review from Copilot September 3, 2025 14:16
Copy link

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

This PR deprecates three API endpoints by replacing their implementation with HTTP 410 Gone responses, indicating these endpoints are no longer available.

  • Deprecates GET /words endpoint that previously returned all word data
  • Deprecates GET /list/user/{userId} endpoint that returned words for a specific user
  • Deprecates GET /list/word/{word} endpoint that returned users viewing a specific word

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
server/oapi/words.go Replaces GetWords implementation with HTTP 410 Gone response
server/oapi/list.go Replaces GetListUserUserId and GetListWordWord implementations with HTTP 410 Gone responses

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Pugma Pugma merged commit c9e2219 into main Sep 3, 2025
7 checks passed
@Pugma Pugma deleted the deprecate-get-all-words branch September 3, 2025 16:51
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.

4 participants