Skip to content

[Manual Sync Required] Fix codes API response data is null#1171

Merged
pulltheflower merged 2 commits into
mainfrom
manual_sync_mr_2561_b15e0c2
Jun 15, 2026
Merged

[Manual Sync Required] Fix codes API response data is null#1171
pulltheflower merged 2 commits into
mainfrom
manual_sync_mr_2561_b15e0c2

Conversation

@csg-pr-bot

Copy link
Copy Markdown
Collaborator

Summary:

  • Fixes the /api/v1/codes endpoint to return an empty array (data: []) instead of null when no code data is found, ensuring consistent API response formatting.
  • Addresses issue #1199 by standardizing the response structure for empty search results.

Manual handling required:

  • Dev-agent failed to cherry-pick this GitLab MR: PR创建失败: 分支推送失败: fatal: unable to access 'https://github.com/OpenCSGs/csghub-server.git/': Failed to connect to github.com port 443 after 130420 ms: Connection timed out

  • Source GitLab MR: !2561

  • Source ref: b97be6ed64fb2c1741dddcef1df4d00dfa7f2854

  • Files in sync scope: 2

  • The GitLab MR patch was applied with git's 3-way apply. Please review before merging.

@pulltheflower pulltheflower force-pushed the manual_sync_mr_2561_b15e0c2 branch from 7058a34 to 5128f6d Compare June 15, 2026 02:56

@pulltheflower pulltheflower left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@pulltheflower pulltheflower merged commit d87970f into main Jun 15, 2026
7 checks passed
@pulltheflower pulltheflower deleted the manual_sync_mr_2561_b15e0c2 branch June 15, 2026 03:08
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