Skip to content

Conversation

floherzler
Copy link

@floherzler floherzler commented Sep 9, 2025

What does this PR do?

Shows the correct tab description pagination (e.g. "48 Buckets per page" instead of "48 Databases"), likely caused by a copy and paste mistake when duplicating the Storage tab from the Databases tab..

Test Plan

This only changes one string

Related PRs and Issues

none

Have you read the Contributing Guidelines on issues?

I have read the Contributing Guidelines.

Summary by CodeRabbit

  • Bug Fixes
    • Corrected pagination label on the Storage page to display “Buckets” instead of “Databases,” aligning the UI with the content being viewed. Pagination behavior (limit, offset, total) remains unchanged.

Copy link

appwrite bot commented Sep 9, 2025

Console

Project ID: 688b7bf400350cbd60e9

Sites (2)
Site Status Logs Preview QR
 console-qa
688b7cf6003b1842c9dc
Failed Failed Authorize Preview URL QR Code
 console-cloud
688b7c18002b9b871a8f
Failed Failed Authorize Preview URL QR Code

Note

Cursor pagination performs better than offset pagination when loading further pages.

Copy link
Contributor

coderabbitai bot commented Sep 9, 2025

Walkthrough

The change modifies src/routes/(console)/project-[region]-[project]/storage/+page.svelte by updating the label prop of the PaginationWithLimit component from "Databases" to "Buckets". The component’s other props (limit, offset, total) continue to bind to data.limit, data.offset, and data.buckets.total respectively. No additional logic, data bindings, or exported/public entity declarations were altered.

Pre-merge checks (3 passed)

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title clearly and concisely summarizes the primary change by stating that the pagination label in the storage tab will display “Buckets” instead of “Databases,” directly reflecting the scope of the refactor.
Description Check ✅ Passed The description directly addresses the change by explaining the incorrect pagination string fix, outlines the specific UI update, and mentions the minimal test plan, making it clearly relevant to the changeset.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

  • Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
  • Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.


📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5b98696 and 17b09ff.

📒 Files selected for processing (1)
  • src/routes/(console)/project-[region]-[project]/storage/+page.svelte (1 hunks)
🔇 Additional comments (1)
src/routes/(console)/project-[region]-[project]/storage/+page.svelte (1)

62-66: LGTM: pagination resource updated to “Buckets”.
Only the /routes/(console)/project-[region]-[project]/databases/+page.svelte file still uses name="Databases", which is expected.

✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant