Skip to content

Add Docs Expo Integration (expo-sqlite & op-sqlite) #84

New issue

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

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

Already on GitHub? # to your account

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

Conversation

austinm911
Copy link

@austinm911 austinm911 commented Feb 20, 2025

Copy link

vercel bot commented Feb 20, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
zero-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 20, 2025 7:46am

@samducker
Copy link

LGTM

@austinm911 austinm911 changed the title Add Docs Expo Integration Add Docs Expo Integration (expo-sqlite & op-sqlite) Feb 20, 2025
Copy link
Contributor

@arv arv left a comment

Choose a reason for hiding this comment

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

LGTM

This is very useful... going to look at the mono PR now.

[userId, authToken],
);

if (!z) {
Copy link
Contributor

Choose a reason for hiding this comment

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

When can this be null?

Copy link
Author

Choose a reason for hiding this comment

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

good point - on the other hand do you want this to be simplified and exclude using the useMemo in the docs (but we can keep the comment about in production it should be memoized). Elsewhere in the docs it is kept simple.

href: string;
noLink?: true;
items?: EachRoute[];
title: string;
Copy link
Contributor

Choose a reason for hiding this comment

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

formatting change seems a bit odd.

maybe run npm run format

@tantaman
Copy link
Contributor

tantaman commented Apr 8, 2025

@aboodman - do we want to go ahead and merge this? Looks like it got stuck because the formatting of routes-config.ts was never fixed.

@arv
Copy link
Contributor

arv commented Apr 8, 2025

I was planning to look at this in deeper details (maybe next week or after the vacation end of the month)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants