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

feat(api): Add preview endpoint #6648

Open
wants to merge 4 commits into
base: next
Choose a base branch
from
Open

feat(api): Add preview endpoint #6648

wants to merge 4 commits into from

Conversation

tatarco
Copy link
Contributor

@tatarco tatarco commented Oct 8, 2024

What changed? Why was the change needed?

We need a quick stateless preview endpoint in order to have a single point or rendering for all workflows type.
what was done?

  1. full flow controller -> hydration -> validation -> render for the preview endpoint.
  2. move new V2 DTO's to a shared-internal package allowing the creation of clients for the FE to use as well as out E2E

https://www.loom.com/share/a3041ae59a9843bf98bb98829ca12a56?sid=6ac0f038-6c1e-4d5c-81c1-2761b7df5977

Screenshots

Expand for optional sections

Related enterprise PR

Special notes for your reviewer

Copy link

netlify bot commented Oct 8, 2024

Deploy Preview for novu-stg-vite-dashboard-poc failed. Why did it fail? →

Name Link
🔨 Latest commit 8a5e0a8
🔍 Latest deploy log https://app.netlify.com/sites/novu-stg-vite-dashboard-poc/deploys/671668bd28431a0008da34f2

Copy link

pkg-pr-new bot commented Oct 8, 2024

Open in Stackblitz

novu

pnpm add https://pkg.pr.new/novuhq/novu@6648

@novu/client

pnpm add https://pkg.pr.new/novuhq/novu/@novu/client@6648

@novu/headless

pnpm add https://pkg.pr.new/novuhq/novu/@novu/headless@6648

@novu/framework

pnpm add https://pkg.pr.new/novuhq/novu/@novu/framework@6648

@novu/nest

pnpm add https://pkg.pr.new/novuhq/novu/@novu/nest@6648

@novu/js

pnpm add https://pkg.pr.new/novuhq/novu/@novu/js@6648

@novu/nextjs

pnpm add https://pkg.pr.new/novuhq/novu/@novu/nextjs@6648

@novu/node

pnpm add https://pkg.pr.new/novuhq/novu/@novu/node@6648

@novu/notification-center

pnpm add https://pkg.pr.new/novuhq/novu/@novu/notification-center@6648

@novu/providers

pnpm add https://pkg.pr.new/novuhq/novu/@novu/providers@6648

@novu/react

pnpm add https://pkg.pr.new/novuhq/novu/@novu/react@6648

@novu/shared

pnpm add https://pkg.pr.new/novuhq/novu/@novu/shared@6648

@novu/react-native

pnpm add https://pkg.pr.new/novuhq/novu/@novu/react-native@6648

@novu/stateless

pnpm add https://pkg.pr.new/novuhq/novu/@novu/stateless@6648

commit: 8a5e0a8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants