Skip to content
This repository was archived by the owner on Sep 21, 2026. It is now read-only.

Disable button when request is active - #2900

Merged
HarshMN2345 merged 1 commit into
mainfrom
ser-355
Mar 2, 2026
Merged

HarshMN2345 merged 1 commit into
mainfrom
ser-355

Conversation

@hmacr

@hmacr hmacr commented Mar 2, 2026 •

Copy link
Copy Markdown
Contributor

Related SER-355

Summary by CodeRabbit

  • Bug Fixes
    • Disabled the Approve Deployment button while a deployment is in progress to prevent accidental multiple submissions.

@appwrite

appwrite Bot commented Mar 2, 2026 •

Copy link
Copy Markdown

Console (appwrite/console)

Project ID: 688b7bf400350cbd60e9

Sites (1)
Site Status Logs Preview QR
 console-stage
688b7cf6003b1842c9dc
Ready Ready View Logs Preview URL QR Code

Tip

Functions can run for up to 15 minutes before timing out

@coderabbitai

coderabbitai Bot commented Mar 2, 2026 •

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0c763d7 and 57921bf.

📒 Files selected for processing (1)
  • src/routes/(authenticated)/git/authorize-contributor/+page.svelte

Walkthrough

A single-line change to src/routes/(authenticated)/git/authorize-contributor/+page.svelte adds a disabled={loading} prop binding to the Approve Deployment button. This disables the button during the deployment approval process by linking the button's disabled state to the existing loading state variable, which is already managed by the approveDeployment function.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ 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 'Disable button when request is active' directly describes the main change: adding a disabled={loading} prop to prevent button interaction during an active deployment operation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ser-355

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.

@HarshMN2345
HarshMN2345 merged commit 2e6440c into main Mar 2, 2026
4 checks passed
@HarshMN2345
HarshMN2345 deleted the ser-355 branch March 2, 2026 19:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants