Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Chore: Add OpenReplay to docs #828

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

Conversation

robertjdominguez
Copy link
Collaborator

@robertjdominguez robertjdominguez commented Dec 30, 2024

Description 📝

Adds OpenReplay to docs using the React package and instructions from their site.

I used Varun's console code as inspiration but tweaked things to our needs with Docusaurus. Notably:

  • Adding envs to the docusaurus.config.ts
  • Passing them directly to the initOpenReplay function instead of as args
  • Calling both the initialization and starting functions as useEffect() with their own depdencies.

I wasn't able to see my sessions in the OpenReplay console, but I imagine that's because we've disabled those for localhost. Either way, when testing locally, I received a 200 from <OUR INGEST ADDRESS>/v1/web/start

Before merging, we'll need to add the envs as key-values for GCP:

  • OPENREPLAY_INGEST_POINT
  • OPENREPLAY_PROJECT_KEY

Adds OpenReplay to docs using the React package and instructions from their site.

I used Varun's console code as inspiration, but tweaked things to our needs
with Docusaurus. Notably:
- Adding envs to the `docusaurus.config.ts`
- Passing them directly to the `initOpenReplay` function instead of as args
- Calling both the initialization and starting functions as `useEffect()` with their
own depdencies.

I wasn't able to see my sessions in the OpenReplay console, but I'm imagining that's because
we've disabled those? Either way, when testing locally, I received a `200` from
`<OUR INGEST ADDRESS>/v1/web/start`

Before merging, we'll need to add the envs as key-values for GCP:
- OPENRELAY_INGEST_POINT
- OPENRELAY_PROJECT_KEY
@hasura-bot
Copy link
Contributor

@robertjdominguez Thanks for your PR! I've assigned @robertjdominguez to review it.

Copy link

Deploying ddn-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: a44330f
Status: ✅  Deploy successful!
Preview URL: https://8c777b9f.v3-docs-eny.pages.dev
Branch Preview URL: https://rob-chore-add-openreplay.v3-docs-eny.pages.dev

View logs

Copy link

@ManushiKhanna ManushiKhanna left a comment

Choose a reason for hiding this comment

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

lgtm

@seanparkross seanparkross self-requested a review January 8, 2025 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants