Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: update solid example to use switch/match and get account on page load #1630

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

bittere
Copy link

@bittere bittere commented Jan 10, 2025

What does this PR do?

This commit updates the SolidJS example in the quickstart to:

  • Use the <Switch>/<Match> syntax instead of if...else conditionals
  • Gets the account when the page loads (in case a user is signed in already)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

I do not think any tests are required; I have verified that this works

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

I cannot find any issues relating to this particular example in the quickstart

Have you read the Contributing Guidelines on issues?

Yes (all relevant parts).

…e load

This commit updates the SolidJS example in the quickstart to:

- Use the `<Switch>/<Match>` syntax instead of `if...else` conditionals
- Gets the account when the page loads (in case a user is signed in already)
@coolify-appwrite-org
Copy link

coolify-appwrite-org bot commented Jan 13, 2025

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2025-01-14 04:20:38 CET

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