feat(react-journey): add react journey app#86
Merged
Conversation
ryanbas21
reviewed
Jan 12, 2026
| ## Requirements | ||
|
|
||
| 1. An instance of Ping's Access Manager (AM), either within a Ping's Advanced Identity Cloud tenant, your own private installation or locally installed on your computer | ||
| 2. Node >= 14.2.0 (recommended: install via [official package installer](https://nodejs.org/en/)) |
Contributor
There was a problem hiding this comment.
maybe we should update this to a newer node, we dont test this far back so maybe 18? 18 is still pretty old
Contributor
Author
There was a problem hiding this comment.
updated to 18.12.0
ryanbas21
reviewed
Jan 12, 2026
| */ | ||
|
|
||
| // TODO: Have we ported over HttpClient yet? | ||
| import { HttpClient } from '@forgerock/javascript-sdk'; |
Contributor
There was a problem hiding this comment.
We haven't but i think we could probably remove this and use a plain fetch
ancheetah
force-pushed
the
SDKS-4047-react-journey
branch
2 times, most recently
from
cc7c400 to
3962b82
Compare
ancheetah
force-pushed
the
SDKS-4047-react-journey
branch
5 times, most recently
from
b09f40e to
315978c
Compare
Contributor
Author
|
Please review "SDK Integration" comments and OIDC Context |
ryanbas21
previously approved these changes
Jan 20, 2026
cerebrl
previously approved these changes
Jan 20, 2026
Contributor
cerebrl
left a comment
cerebrl
left a comment
There was a problem hiding this comment.
This looks good. I have one non-blocking that we can address in a follow up PR.
| * @function useOidcClient - A custom hook to access the OIDC client from React Context | ||
| * @returns - An OIDC client if it exists otherwise an error | ||
| */ | ||
| export function useOidcClient() { |
Contributor
There was a problem hiding this comment.
It seems this module differs a bit from the global-state.js pattern. Can we make sure these are consistent in both file name and internal patterns?
See the screenshot attached.
| * | ||
| * protect.context.js | ||
| * | ||
| * Copyright (c) 2026 Ping Identity Ping Identity Corporation. All rights reserved. |
Contributor
There was a problem hiding this comment.
Hey just a heads up: I noticed "Ping Identity" is showing up twice in the copyright header of a few files.
Contributor
Author
There was a problem hiding this comment.
Good catch, I'll update these in my next pr. Thanks!
ancheetah
force-pushed
the
SDKS-4047-react-journey
branch
from
4e8160b to
edbf135
Compare
vatsalparikh
reviewed
Mar 7, 2026
ancheetah
force-pushed
the
SDKS-4047-react-journey
branch
2 times, most recently
from
875d81b to
2797407
Compare
cerebrl
previously approved these changes
Apr 1, 2026
SteinGabriel
reviewed
Apr 1, 2026
ancheetah
force-pushed
the
SDKS-4047-react-journey
branch
from
2797407 to
ae40810
Compare
SteinGabriel
reviewed
Apr 1, 2026
ancheetah
force-pushed
the
SDKS-4047-react-journey
branch
from
ae40810 to
a3e9ff7
Compare
SteinGabriel
previously approved these changes
Apr 1, 2026
SteinGabriel
approved these changes
Apr 1, 2026
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
5 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.
Jira: https://pingidentity.atlassian.net/browse/SDKS-4047
TODO: