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

docs: Update PowerSync recipe with latest content #1

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

Conversation

stevensJourney
Copy link
Collaborator

@stevensJourney stevensJourney commented Sep 4, 2024

Overview

This will eventually update the local first recipe for PowerSync. This PR exists here first for internal feedback and discussion.

The recipe was quite outdated.

Work done:

  • Removed polyfill installation steps and Flipper network inspector step
  • Added Metro config step for inline requires
  • Fixed minor inconsistencies in code blocks. Some steps include importing code which is only created much later in the guide.
  • Updated hooks to use useQuery instead of the deprecated usePowerSyncQuery
  • Removed the need for the expo-crypto package by using built-in uuid() method
  • Fixed Typos in table name constants

This was tested by following the guide. It would be nice if a reviewer could double check this on their end.

@stevensJourney stevensJourney marked this pull request as ready for review September 4, 2024 08:43
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