Skip to content

Conversation

@madhav165
Copy link
Collaborator

🐛 Bug-fix PR


📌 Summary

Fixed the following pagination issues in Server page

  1. Fixed Select All so newly loading prompts and resources are selected as they load.
  2. Pagination added in Edit Server screen for prompts and resources.

These were already implemented for tools in an earlier PR.

🧪 Verification

Check Command Status
Lint suite make lint pass
Unit tests make test pass

📐 MCP Compliance (if relevant)

  • Matches current MCP spec
  • No breaking change to MCP clients

✅ Checklist

  • Code formatted (make black isort pre-commit)
  • No secrets/credentials committed

Signed-off-by: Madhav Kandukuri <[email protected]>
Signed-off-by: Madhav Kandukuri <[email protected]>
Signed-off-by: Madhav Kandukuri <[email protected]>
@madhav165 madhav165 self-assigned this Nov 10, 2025
@rakdutta rakdutta self-requested a review November 10, 2025 16:09
Copy link
Collaborator

@rakdutta rakdutta left a comment

Choose a reason for hiding this comment

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

Works as expected. If 'Select All' is used, it selects all prompts and resources, including those not currently present in the DOM.

@crivetimihai crivetimihai merged commit c7c1c94 into main Nov 11, 2025
45 checks passed
@crivetimihai crivetimihai deleted the fix-pagination-bugs branch November 11, 2025 08:33
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