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: OpenApi Parser multiple success responses, error response descriptions, better examples #1913

Closed
wants to merge 29 commits into from

Conversation

RohinBhargava
Copy link
Member

Short description of the changes made

  • Focuses heavily on examples, with multiple success/failure responses
  • Changes FDR EndpointDefinition to accept multiple requests and responses, now
  • Prefers first request and response in UI for now
  • Plumbs redoc examples
  • Ensures enums are plumbed to errors

What's not included:

  • Updated UI
  • Formats and Regexes
  • Allowing for curl snippet to be processed

What was the motivation & context behind this PR?

  • Customer Go-Live

How has this PR been tested?

Screen.Recording.2024-12-19.at.4.32.56.AM.mov

Copy link

vercel bot commented Dec 19, 2024

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

Name Status Preview Updated (UTC)
app-dev.buildwithfern.com ❌ Failed (Inspect) Dec 20, 2024 5:43pm
app-slash.ferndocs.com ❌ Failed (Inspect) Dec 20, 2024 5:43pm
app.buildwithfern.com ❌ Failed (Inspect) Dec 20, 2024 5:43pm
app.ferndocs.com ❌ Failed (Inspect) Dec 20, 2024 5:43pm
icons.ferndocs.com ❌ Failed (Inspect) Dec 20, 2024 5:43pm
rbac.ferndocs.com ❌ Failed (Inspect) Dec 20, 2024 5:43pm
1 Skipped Deployment
Name Status Preview Updated (UTC)
fern-shell ⬜️ Ignored (Inspect) Dec 20, 2024 5:43pm

@RohinBhargava RohinBhargava had a problem deploying to Preview - app.buildwithfern.com December 19, 2024 04:36 — with GitHub Actions Failure
@RohinBhargava RohinBhargava had a problem deploying to Preview - app-dev.buildwithfern.com December 19, 2024 04:36 — with GitHub Actions Failure
Copy link

Copy link

Copy link

Copy link

Copy link

Copy link

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.

4 participants