-
Notifications
You must be signed in to change notification settings - Fork 205
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
fix: internal schema apprears as object on logout #2705
Conversation
✅ Deploy Preview for stoplight-elements ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for stoplight-elements-demo ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Please add the screenshot for the other usecase like the dropdown contains all the error messages/ showing an Object values |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me., though I have added on comment. Please address it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
going to need some test coverage for this
}; | ||
expect(resolvedObject).toEqual(filteredObject); | ||
}); | ||
it('show the error if all object contains an error for oneOf', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
may be you can put a comment saying "The schema marked as internal so it's populate the error object.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes looks to good to me
STOP-1027
In general, make sure you have: (check the boxes to acknowledge you've followed this template)
Previously, if not logged in, the schema which is marked internal showing up in dropdown as an empty 'object' for oneOf/anyOf in public docs.
Before
After
In case of every schema is internal in the list it will display the error message.
CONTRIBUTING.md
Other Available PR Templates:
CONTRIBUTING.md