Skip to content

Conversation

@shanejonas
Copy link
Member

@shanejonas shanejonas commented May 7, 2019

fixes #71
fixes #72
fixes #73

complex array items


image

nested objects


image

better one of support


image

@codecov
Copy link

codecov bot commented May 7, 2019

Codecov Report

Merging #74 into master will increase coverage by 0.94%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #74      +/-   ##
==========================================
+ Coverage   91.25%   92.19%   +0.94%     
==========================================
  Files          16       16              
  Lines         343      346       +3     
  Branches       81       82       +1     
==========================================
+ Hits          313      319       +6     
+ Misses         16       13       -3     
  Partials       14       14
Impacted Files Coverage Δ
src/JSONSchema/JSONSchema.tsx 87.5% <100%> (+12.5%) ⬆️
src/JSONSchema/fields/JSONSchemaFields.tsx 86.66% <100%> (+1.48%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 09f7b7d...a1e018f. Read the comment docs.

@shanejonas shanejonas added the bug Something isn't working label May 7, 2019
@shanejonas shanejonas self-assigned this May 7, 2019
Copy link
Member

@zcstarr zcstarr left a comment

Choose a reason for hiding this comment

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

🐑 lgtm. One non blocking comment, is that eventually ux wise, if I'm looking at the code correctly we'll need to limit the number of levels nested ( visually) . Or alternatively have an eventual re-think of the ux. I think this patch will make a big difference for people trying to use the playground! Nice!

@zcstarr zcstarr self-requested a review May 7, 2019 20:42
@shanejonas shanejonas merged commit 1f3db30 into master May 7, 2019
@shanejonas shanejonas deleted the fix/json-schema-nested branch May 7, 2019 21:00
@shanejonas
Copy link
Member Author

🎉 This PR is included in version 1.0.9 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

bug Something isn't working released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

items array doesnt render complex objects nested object only shows "object" oneOf only works in certain cases

3 participants