Skip to content

Conversation

@thatrobotdev
Copy link

  • Adds a YAML to Zod Schema transformer by first converting the YAML to JSON using the same method as the YAML to JSON transformer, then converting the JSON to Zod Schema using the same method as the JSON to Zod Schema transformer.
  • Creates a new YAML category in the sidebar

Problems with this PR ;)

The implementation in this PR works, but has some minor flaws that I need help fixing! It could be merged as is with these caveats:

  • Doesn't define packageName or packageUrl in routes.tsx because it uses both the packages json-to-zod and yaml.
    • To fix: Add a way to credit two packages at once, or implement a package that does a direct YAML to Zod Schema conversion.
  • Reuses code from YamlToJson() and JsonToZod() without importing them.
    • To fix: Importing the functions YamlToJson() and JsonToZod() directly instead of being made up of their code.

- Adds a YAML to Zod Schema transformer by first converting the YAML to JSON using the same method as the YAML to JSON transformer, then converting the JSON to Zod Schema using the same method as the JSON to Zod Schema transformer.
- Creates a new YAML category in the sidebar
@vercel
Copy link

vercel bot commented Sep 2, 2023

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

Name Status Preview Comments Updated (UTC)
transform ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 17, 2024 10:06am

Comment on lines -390 to -391
label: "TOML to YAML",
path: "/toml-to-yaml"
Copy link
Owner

Choose a reason for hiding this comment

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

This has been removed ?

@dirtycajunrice
Copy link
Collaborator

Hey! Thanks for the pull request!
Heads up this is just a friendly notice to let you know I will be helping out with the project and reviewing approvals. I am currently refactoring the app to nextjs 14, so modifications of your PR will be needed.

I will message again when that time has come 😄

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