chore: update dependencies and improve README for React authentication example#2356
Open
chore: update dependencies and improve README for React authentication example#2356
Conversation
wassimoo
requested review from
aeneasr,
piotrmsc,
vinckr and
zepatrik
as code owners
vinckr
requested changes
Nov 24, 2025
Member
vinckr
left a comment
vinckr
left a comment
There was a problem hiding this comment.
Nice, thanks!
a few small formatting etc nits to resolve then can be merged imo
|
|
||
| This project demonstrates how to integrate Ory authentication into a React | ||
| application built with Vite and TypeScript. | ||
| This example demonstrates how to implement authentication in a React.js |
Member
There was a problem hiding this comment.
Suggested change
| This example demonstrates how to implement authentication in a React.js | |
| This example demonstrates how to implement authentication in a React |
nit but I think the official name is React, also see https://react.dev/
| - An Ory Network account with a project set up | ||
| - Your Ory Project ID | ||
|
|
||
| ## Getting Started |
Member
There was a problem hiding this comment.
Suggested change
| ## Getting Started | |
| ## Get started |
please always use sentence casing and active form in headings - see README.md
|
|
||
| ## Getting Started | ||
|
|
||
| ### 1. Install Dependencies |
Member
There was a problem hiding this comment.
Suggested change
| ### 1. Install Dependencies | |
| ### 1. Install dependencies |
| ``` | ||
|
|
||
| # Start the development server | ||
| ### 2. Start the Application |
Member
There was a problem hiding this comment.
Suggested change
| ### 2. Start the Application | |
| ### 2. Start the application |
| used as the `baseUrl` in the Ory SDK configuration. | ||
| used as the `basePath` in the Ory SDK configuration. | ||
|
|
||
| ## Making API Calls |
Member
There was a problem hiding this comment.
Suggested change
| ## Making API Calls | |
| ## Making API calls |
| @@ -54,7 +68,8 @@ Ory to authenticate requests. | |||
| ## Project Structure | |||
Member
There was a problem hiding this comment.
Suggested change
| ## Project Structure | |
| ## Project structure |
| - `src/AppWithApi.tsx` - Authentication with API integration example | ||
| - `src/api/index.js` - Express.js API server example | ||
|
|
||
| ## Available Scripts |
Member
There was a problem hiding this comment.
Suggested change
| ## Available Scripts | |
| ## Available scripts |
|
|
||
| This guide shows how to create a simple React application and secure it with authentication powered by Ory. You can use this guide | ||
| with both Ory Network and self-hosted Ory software. | ||
| This guide shows how to create a simple React native application and secure it with authentication powered by Ory. You can use |
Member
There was a problem hiding this comment.
Suggested change
| This guide shows how to create a simple React native application and secure it with authentication powered by Ory. You can use | |
| This guide shows how to create a simple React Native application and secure it with authentication powered by Ory. You can use |
Comment on lines
+67
to
+68
| `App.tsx` code below). The `--dev` flag disables security checks for easier integration and should not be used when running the | ||
| Tunnel in an insecure environment like public networks. |
Member
There was a problem hiding this comment.
Suggested change
| `App.tsx` code below). The `--dev` flag disables security checks for easier integration and should not be used when running the | |
| Tunnel in an insecure environment like public networks. | |
| `App.tsx` code below). The `--dev` flag disables security checks for easier integration and shouldn't be used when running Ory Tunnel in an insecure environment like public networks. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related Issue or Design Document
Checklist
If this pull request addresses a security vulnerability,
I confirm that I got approval (please contact security@ory.com) from the maintainers to push the changes.
Further comments