Skip to content

Conversation

ItzNotABug
Copy link
Member

@ItzNotABug ItzNotABug commented Sep 30, 2025

What does this PR do?

Self-hosted options for sign up, misc. design changes.

Test Plan

Manual.

Related PRs and Issues

N/A.

Have you read the Contributing Guidelines on issues?

Yes.

Summary by CodeRabbit

  • New Features

    • Context-aware title and subtitle that reflect cloud availability.
    • Suggestions toggle appears only when the feature is active.
    • Automatically enables suggestions on load in cloud environments.
  • Improvements

    • Adjusted layout spacing and alignment based on cloud state for a cleaner UI.
    • Simplified Sign up button with a clearer external link.
    • Clearer guidance for non-cloud environments.
  • Refactor

    • Removed obsolete UI references and streamlined dependencies.
    • Adopted standardized UI components for consistency.

@ItzNotABug ItzNotABug self-assigned this Sep 30, 2025
Copy link

appwrite bot commented Sep 30, 2025

Console

Project ID: 688b7bf400350cbd60e9

Sites (2)
Site Status Logs Preview QR
 console-stage
688b7cf6003b1842c9dc
Building Building View Logs Preview URL QR Code
 console-cloud
688b7c18002b9b871a8f
Ready Ready View Logs Preview URL QR Code

Note

Appwrite has a Discord community with over 16 000 members.

Copy link
Contributor

coderabbitai bot commented Sep 30, 2025

Walkthrough

Updates input.svelte to conditionally render and style based on cloud featureActive. Introduces derived title/subtitle values, replaces hard-coded heading, and shows the suggestions toggle only when featureActive is true. Retains non-cloud Sign up flow with a simplified external link button (icon removed). Adds onMount to enable tableColumnSuggestions when featureActive is true. Removes isSmallViewport and IconExternalLink imports; adds onMount and isCloud; uses pink-svelte components directly. Keeps a guarded block for input context when suggestions and featureActive are true. No exported/public API changes.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title accurately summarizes the primary change by indicating that the suggestions input component’s design has been updated. It is concise, focuses on the main UI adjustments in input.svelte, and avoids unnecessary details. Teammates scanning the history can immediately understand that this PR concerns styling and layout changes to the suggestions input.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch design-updates

📜 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 05befa8 and 91728ac.

📒 Files selected for processing (1)
  • src/routes/(console)/project-[region]-[project]/databases/database-[database]/(suggestions)/input.svelte (2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: e2e
  • GitHub Check: build
🔇 Additional comments (8)
src/routes/(console)/project-[region]-[project]/databases/database-[database]/(suggestions)/input.svelte (8)

2-3: LGTM! Import additions are appropriate.

The new imports (onMount and isCloud) are properly utilized in the component logic.


10-14: LGTM! Initialization logic is correct.

The onMount hook properly enables suggestions for cloud environments.


16-28: LGTM! Derived values provide clear conditional messaging.

The title and subtitle appropriately adapt based on whether the feature is available (cloud vs. self-hosted).


32-43: LGTM! Layout adjustments enhance the visual hierarchy.

The conditional gap spacing and use of the dynamic title variable improve the component's adaptability.


45-52: LGTM! Conditional switch rendering is appropriate.

The suggestions toggle correctly appears only for cloud environments where the feature is active.


55-61: LGTM! Sign up flow is clear and functional.

The button correctly directs self-hosted users to the cloud registration page.


63-74: LGTM! Input context section is properly guarded.

The double condition ensures the textarea only appears when both the feature is available and the user has enabled it.


77-81: LGTM! Style enhances button interactivity.

The cursor pointer style appropriately indicates clickable state for the switch button.


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
🧪 Early access (Sonnet 4.5): enabled

We are currently testing the Sonnet 4.5 model, which is expected to improve code review quality. However, this model may lead to increased noise levels in the review comments. Please disable the early access features if the noise level causes any inconvenience.

Note:

  • Public repositories are always opted into early access features.
  • You can enable or disable early access features from the CodeRabbit UI or by updating the CodeRabbit configuration file.

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

@ItzNotABug ItzNotABug merged commit 05052aa into main Sep 30, 2025
4 of 5 checks passed
@ItzNotABug ItzNotABug deleted the design-updates branch September 30, 2025 07:54
@coderabbitai coderabbitai bot mentioned this pull request Oct 1, 2025
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