Skip to content

try flask_restful -> flask_restx#24310

Merged
laipz8200 merged 4 commits intolanggenius:mainfrom
asukaminato0721:restx
Aug 24, 2025
Merged

try flask_restful -> flask_restx#24310
laipz8200 merged 4 commits intolanggenius:mainfrom
asukaminato0721:restx

Conversation

@asukaminato0721
Copy link
Copy Markdown
Contributor

@asukaminato0721 asukaminato0721 commented Aug 21, 2025

Important

  1. Make sure you have read our contribution guidelines
  2. Ensure there is an associated issue and you have been assigned to it
  3. Use the correct syntax to link this PR: Fixes #<issue number>.

part of #24173

Summary

This is only a test, to try to update something.

this can have auto swagger doc generation. https://flask-restx.readthedocs.io/en/latest/swagger.html

Screenshots

Before After
... ...

Checklist

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

@laipz8200 laipz8200 self-assigned this Aug 21, 2025
@asukaminato0721 asukaminato0721 marked this pull request as ready for review August 21, 2025 19:39
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Aug 21, 2025
@laipz8200 laipz8200 self-requested a review August 22, 2025 06:47
@laipz8200
Copy link
Copy Markdown
Member

merged into our development environment for testing.

@laipz8200
Copy link
Copy Markdown
Member

I think this change has been tested. Would you be willing to rebase main and verify it's up to date?

Signed-off-by: -LAN- <laipz8200@outlook.com>
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 24, 2025
@laipz8200 laipz8200 merged commit 18dce66 into langgenius:main Aug 24, 2025
9 checks passed
@asukaminato0721 asukaminato0721 deleted the restx branch August 24, 2025 07:01
Eric-Guo added a commit to Eric-Guo/dify that referenced this pull request Aug 24, 2025
qiaofenlin pushed a commit to qiaofenlin/dify that referenced this pull request Aug 24, 2025
Signed-off-by: -LAN- <laipz8200@outlook.com>
Co-authored-by: -LAN- <laipz8200@outlook.com>
@lyzno1 lyzno1 mentioned this pull request Aug 25, 2025
qiqizjl pushed a commit to qiqizjl/dify that referenced this pull request Aug 27, 2025
Signed-off-by: -LAN- <laipz8200@outlook.com>
Co-authored-by: -LAN- <laipz8200@outlook.com>
HarryReidx pushed a commit to HarryReidx/dify that referenced this pull request Sep 1, 2025
Signed-off-by: -LAN- <laipz8200@outlook.com>
Co-authored-by: -LAN- <laipz8200@outlook.com>
(cherry picked from commit 18dce66)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants