Skip to content

docs: add expo setup instructions to installation guide#1853

Open
Undertone0809 wants to merge 1 commit intoNozbe:masterfrom
Undertone0809:master
Open

docs: add expo setup instructions to installation guide#1853
Undertone0809 wants to merge 1 commit intoNozbe:masterfrom
Undertone0809:master

Conversation

@Undertone0809
Copy link
Copy Markdown

No description provided.

Comment thread docs-website/docs/docs/Installation.mdx Outdated
api.cache(true);

return {
presets: [['babel-preset-expo', { jsxImportSource: 'nativewind' }], 'nativewind/babel'],
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I don't think nativewind is related/required for watermelon install? Can you check this?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Sorryyyy for that. Now I use default expo config.

@Darhagonable
Copy link
Copy Markdown

This expo plugin already exist: https://github.com/morrowdigital/watermelondb-expo-plugin
WatermelonDB should just link to it for Expo installation instructions.

@Obhenimen
Copy link
Copy Markdown

The expo plugin library doesn't support it anymore. It has not been updated in a long time.

api.cache(true);

return {
presets: [['babel-preset-expo'], 'nativewind/babel'],
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Either the PR is incomplete or it doesn't work because just having babel plugins won't make WatermelonDB work on Expo as there's no SQLite available unless you use Expo SQLite plugin.

The preferable approach should be a new adopted SQLiteExpo under the adapters that operates on SQLite via Expo and rest of the machinery should work out of the box. For the web, could use sql.js

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.

5 participants