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

#4546 - should return the same valueObj for the same objPath #4548

Closed
wants to merge 5 commits into from
Closed

#4546 - should return the same valueObj for the same objPath #4548

wants to merge 5 commits into from

Conversation

kilbot
Copy link
Sponsor Contributor

@kilbot kilbot commented Mar 11, 2023

This PR contains:

An illustration of an issue I am having with nested schemas. The problem relates to these lines of code. I'm unsure if it's possible, or even desirable to memoize the valueObj. It could just be a hole in my understanding.

Describe the problem you have without this PR

#4546

Todos

  • Tests
  • Documentation
  • Typings
  • Changelog

@pubkey
Copy link
Owner

pubkey commented Mar 12, 2023

Fixed in the attached PR.

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.

None yet

2 participants