Skip to content

feat(graph): expose functions to render pdv & error page#27833

Merged
jaysoo merged 6 commits intomasterfrom
pdv-rendering
Sep 13, 2024
Merged

feat(graph): expose functions to render pdv & error page#27833
jaysoo merged 6 commits intomasterfrom
pdv-rendering

Conversation

@MaxKless
Copy link
Copy Markdown
Contributor

@MaxKless MaxKless commented Sep 9, 2024

we have fully isolated graph & error components with a good api but no way to access them directly from the outside (in console).

This PR adds two functions to the window object so that we can render the PDV directly instead of needing the entire app with routing and everything.

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 9, 2024

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

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview Sep 13, 2024 4:22pm

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Sep 9, 2024

Comment thread graph/ui-project-details/src/lib/project-details/project-details.tsx Outdated
Comment thread graph/ui-project-details/src/lib/project-details/project-details.tsx Outdated
Comment thread graph/ui-project-details/src/lib/project-details/project-details.tsx Outdated
Comment thread graph/ui-project-details/src/lib/project-info-header/project-info-header.tsx Outdated
Comment thread graph/client/src/main.tsx
@jaysoo
Copy link
Copy Markdown
Member

jaysoo commented Sep 13, 2024

I tested this on latest and canary Nx releases. Can verify that everything is working with the old window.externalApi.openProjectDetails(...) flow and the new renderPDV(...) flow.

@jaysoo jaysoo merged commit dc821ab into master Sep 13, 2024
@jaysoo jaysoo deleted the pdv-rendering branch September 13, 2024 18:48
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 19, 2024
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.

2 participants