fix: pass platform key to wizard when continuing platform setup #2739

suhaniawasthi10 · 2025-12-25T21:39:55Z

Fixes #10873 - Cannot read properties of undefined (reading 'smallIcon')

When clicking 'Continue' on an existing web platform, the wizard was not receiving the framework key, causing selectedFramework to be undefined.

What does this PR do?

Fixes the crash that occurs when clicking "Continue" on an existing web platform in the onboarding flow.

Root Cause: The continuePlatform() function was not passing the framework key to the wizard, causing selectedFramework to be undefined.

Changes:

+page.svelte: Added key parameter to continuePlatform()
onboard.svelte: Pass platform.key when calling the function
createWeb.svelte: Added defensive null check

Test Plan

Create a project with both Web and Android platforms
Go to Project → Overview
Click "Continue" on the Web platform card
Verify the wizard opens without crashing

Ran pnpm run check (0 errors) and pnpm run lint (passed).

Related PRs and Issues

Fixes appwrite/appwrite#10873

Have you read the Contributing Guidelines on issues?

Yes

Summary by CodeRabbit

Bug Fixes
- Framework details section now only displays when a framework is selected, improving clarity during setup.
Refactor
- Enhanced platform wizard initialization to properly handle additional platform metadata.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

Fixes #10873 - Cannot read properties of undefined (reading 'smallIcon') When clicking 'Continue' on an existing web platform, the wizard was not receiving the framework key, causing selectedFramework to be undefined.

appwrite · 2025-12-25T21:39:59Z

Console (appwrite/console)

Project ID: 688b7bf400350cbd60e9

Sites (1)

Site	Status	Logs	Preview	QR
console-stage `688b7cf6003b1842c9dc`	Failed	Authorize	Preview URL

Tip

GraphQL API works alongside REST and WebSocket protocols

coderabbitai · 2025-12-25T21:40:08Z

Walkthrough

The pull request adds a new optional key parameter to the continuePlatform function in the platforms module. When opening a platform wizard with an existing platform, the code now passes the platform.key as a fourth argument to continuePlatform. The function signature and implementation are updated to accept and forward this key parameter to the wizard.start call. Additionally, conditional rendering logic in the createWeb component is tightened to only render the "Details" card when a framework is selected, rather than whenever the framework selection state is not being changed.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

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 describes the main change: passing the platform key parameter to the wizard during continued platform setup, which directly addresses the root cause of the reported crash.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e446f7f and 67121a1.

📒 Files selected for processing (3)

src/routes/(console)/project-[region]-[project]/overview/onboard.svelte
src/routes/(console)/project-[region]-[project]/overview/platforms/+page.svelte
src/routes/(console)/project-[region]-[project]/overview/platforms/createWeb.svelte

🧰 Additional context used

📓 Path-based instructions (5)

**/*.{ts,tsx,js,jsx,svelte}