Skip to content

feat: add pagination for plugin page #20151

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
Merged
Yeuoly merged 1 commit into from
May 27, 2025
Merged

feat: add pagination for plugin page #20151

Yeuoly merged 1 commit into from
May 27, 2025

Conversation

@LeeeeeeM
Copy link
Contributor

@LeeeeeeM LeeeeeeM commented May 23, 2025 •
edited
Loading

Summary

add pagination for plguin page

depend on this pr langgenius/dify-plugin-daemon#302

close #20224

Screenshots

Before After
x 2025-05-23 15 38 54

Checklist

Important

Please review the checklist below before submitting your pull request.

  • 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

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. 💪 enhancement New feature or request labels May 23, 2025
@LeeeeeeM
Copy link
Contributor Author

LeeeeeeM commented May 23, 2025 •
edited
Loading

langgenius/dify-plugin-daemon#302

please merge together if they are all ok

@Yeuoly @crazywoola

@LeeeeeeM LeeeeeeM mentioned this pull request May 23, 2025
Merged
@LeeeeeeM LeeeeeeM force-pushed the feat/pagination-plugin branch 2 times, most recently from 53655ce to d0dd3a4 Compare May 23, 2025 07:50
@LeeeeeeM LeeeeeeM force-pushed the feat/pagination-plugin branch from d0dd3a4 to 57fbaae Compare May 23, 2025 08:00
@LeeeeeeM LeeeeeeM changed the title feat: add pagination for plguin page feat: add pagination for plugin page May 23, 2025
@LeeeeeeM
Copy link
Contributor Author

LeeeeeeM commented May 26, 2025

148_1748243757.mp4

@crazywoola crazywoola requested a review from Yeuoly May 26, 2025 08:26
Yeuoly
Yeuoly approved these changes May 27, 2025
View reviewed changes
Copy link
Contributor

@Yeuoly Yeuoly left a comment

Choose a reason for hiding this comment

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

LGTM

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label May 27, 2025
@Yeuoly Yeuoly merged commit 756f35f into langgenius:main May 27, 2025
7 checks passed
@pgs-bot pgs-bot bot mentioned this pull request May 27, 2025
Merged
jsincorporated pushed a commit to jsincorporated/asaAi that referenced this pull request Jul 8, 2025
@LeeeeeeM @jsincorporated
feat: add pagination for plugin page (langgenius#20151)
d6145b6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Yeuoly Yeuoly Yeuoly approved these changes

Assignees

No one assigned

Labels

💪 enhancement New feature or request lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

If the plugin length exceeds 256, not all plugins can be displayed.

2 participants

@LeeeeeeM @Yeuoly