Skip to content
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

Jump to bottom

add code snippet and update a link #448

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

add code snippet and update a link #448

wants to merge 1 commit into from

Conversation

KennyHuRadar
Copy link
Contributor

@KennyHuRadar KennyHuRadar commented Feb 29, 2024 •
edited
Loading

What?

  • Add in snippet of code to trackOnce each time the app foregrounds.
  • Fix broken link to apple docs.
  • Did not add more details to the battery usage section of RN, asking developers to conditionally mount the js bundle if the app is launching from a location update. This suggestion is involved and requires messing with the app delegate, which is set up different in different versions of RN. I think we should keep our suggestion hand wavy for now.

Why?

How?

Screenshots (optional)

Anything Else? (optional)

@vercel Vercel
Copy link

vercel bot commented Feb 29, 2024

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

Name Status Preview Updated (UTC)
documentation ✅ Ready (Inspect) Visit Preview Feb 29, 2024 7:21pm

@KennyHuRadar KennyHuRadar marked this pull request as ready for review February 29, 2024 19:28
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@lmeier lmeier lmeier approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@KennyHuRadar @lmeier