Skip to content
This repository has been archived by the owner on Sep 22, 2023. It is now read-only.

AMP-67753 #547

Merged
merged 2 commits into from Jan 26, 2023
Merged

Conversation

curiousYi
Copy link
Contributor

@curiousYi curiousYi commented Jan 26, 2023

Amplitude Developer Docs PR

Updating link to reactnative doc.

Ran it locally and verified the link worked.

Deadline

When do these changes need to be live on the site?

Change type

  • Doc bug fix. Fixes #[insert issue number]. Amplitude contributors include Jira issue number.
  • Doc update.
  • New documentation.
  • Non-documentation related fix or update.

PR checklist:

  • My documentation follows the style guidelines of this project.
  • I previewed my documentation on a local server using mkdocs serve.
  • Running mkdocs serve didn't generate any failures.
  • I have performed a self-review of my own documentation.

@amplitude-dev-docs
@caseyamp

Copy link
Collaborator

@bgiori bgiori left a comment

Choose a reason for hiding this comment

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

That is the old react-native library. New link should be https://github.com/amplitude/Amplitude-TypeScript/tree/main/packages/analytics-react-native

@curiousYi curiousYi requested a review from bgiori January 26, 2023 22:58
@aws-amplify-us-east-1
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-547.d19s7xzcva2mw3.amplifyapp.com

@curiousYi
Copy link
Contributor Author

@kevinpagtakhan Can you merge? I don't have write access :(.

@kevinpagtakhan kevinpagtakhan merged commit 0a02da5 into Amplitude-Developer-Docs:main Jan 26, 2023
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.

None yet

3 participants