Skip to content

Conversation

@LucDeCaf
Copy link
Contributor

@LucDeCaf LucDeCaf commented Dec 3, 2025

The PowerChat demo (found here) is one of the older React Native demos in the repository. As such, it has naturally become less stable and harder to debug as PowerSync, Expo, React Native and other packages all increase their versions. This naturally leads to wanting to bump the package versions - however, some packages are so far behind that it would frankly be easier to start a new project and copy the application code, while also writing new config files that work with the latest packages.

This PR is a rewrite of the application using the latest compatible package versions and package configurations. Most of the app code is copied and should work the same, but most of the configuration code (Metro config, Babel config, Expo config) is new.

Also see PR #784, which is partially the reason this demo is being updated.

@changeset-bot
Copy link

changeset-bot bot commented Dec 3, 2025

⚠️ No Changeset found

Latest commit: 2bb871d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@LucDeCaf LucDeCaf force-pushed the chore/migrate-powerchat-demo branch from 8892fda to 080f9b3 Compare December 3, 2025 22:44
@LucDeCaf LucDeCaf marked this pull request as ready for review December 4, 2025 07:46
@LucDeCaf LucDeCaf requested a review from Chriztiaan December 4, 2025 07:46
UpdateType,
type PowerSyncCredentials
} from "@powersync/react-native";
import type { SupabaseClient } from "@supabase/supabase-js";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few files have the incorrect string formatting.

"@expo/vector-icons": "^15.0.3",
"@faker-js/faker": "^10.1.0",
"@journeyapps/react-native-quick-sqlite": "^2.4.11",
"@powersync/attachments": "^2.4.1",
Copy link
Contributor

@Chriztiaan Chriztiaan Dec 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Technically this is stealth changing the dropping of the "workspace" value.
Either we use workspace for this PR agains't main, or we target the the other PR's branch from this PR for now while reviewing.

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.

3 participants