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.

Sign up for GitHub

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

Jump to bottom

Default boolean property value not rendered #1779

Closed
cmeeren opened this issue Oct 22, 2021 · 8 comments · Fixed by #1781
Closed

Default boolean property value not rendered #1779

cmeeren opened this issue Oct 22, 2021 · 8 comments · Fixed by #1781
Assignees
@zalesky
Labels
Type: Bug

Comments

@cmeeren
Copy link

cmeeren commented Oct 22, 2021

These properties:

image

Render as:

image

As you can see, the default value is missing.

If I instead use strings for the default value, it works:

image

image
@zalesky
Copy link
Contributor

zalesky commented Oct 22, 2021

@cmeeren Thanks for reporting the issue. I can't reproduce it so could you be so kind to provide more details?

category:
  type: object
  properties:
    bool1:
      type: boolean
      default: true
    bool2:
      type: boolean
      default: false
  required:
    - bool1
    - bool2

Screenshot 2021-10-22 at 18 06 24

@cmeeren
Copy link
Author

cmeeren commented Oct 26, 2021

Forgot to mention that I was rendering a schema using SchemaDefinition (in a tag description if that matters), not a request body. Can you reproduce then?

@zalesky
Copy link
Contributor

zalesky commented Oct 27, 2021

@cmeeren could you provide a definition, please?

@cmeeren
Copy link
Author

cmeeren commented Oct 27, 2021 

test:
  properties:
    test:
      type: boolean
      default: false

image

I am using the redoc version fetched by https://cdn.jsdelivr.net/npm/redoc@next/bundles/redoc.standalone.js

@zalesky
Copy link
Contributor

zalesky commented Oct 27, 2021 •
edited
Loading

@cmeeren sorry, it's not helpful, I still don't know where you put this in the definition. Can you provide a full definition or be more clear where is placed this part?

test:
  properties:
    test:
      type: boolean
      default: false

I see "not a request body" and I've never seen anything like "in a tag description" before

@cmeeren
Copy link
Author

cmeeren commented Oct 27, 2021 

openapi: 3.0.2
info: {}

tags:
  - name: foo
    description: |
      <SchemaDefinition schemaRef="#/components/schemas/test" />

components:

  schemas:

    test:
      properties:
        test:
          type: boolean
          default: false

@zalesky
Copy link
Contributor

zalesky commented Oct 27, 2021

tried next steps without success:

  • create file test.html
<!DOCTYPE html>
<html>
<body>
<redoc spec-url=http://localhost:5000/openapi.yaml></redoc>
<script src="https://cdn.jsdelivr.net/npm/redoc@next/bundles/redoc.standalone.js"></script>
</body>
</html>
  • create file openapi.yaml
openapi: 3.0.2
info: {}
tags:
  - name: foo
    description: |
      <SchemaDefinition schemaRef="#/components/schemas/test" />
components:
  schemas:
    test:
      required:
        -test
      properties:
        test:
          type: boolean
          default: false

Screenshot 2021-10-27 at 14 36 47

@cmeeren
Copy link
Author

cmeeren commented Oct 27, 2021 •
edited
Loading

I see now that it only happens when using the enumSkipQuotes option. Sorry for not catching that sooner. Are you able to reproduce then?

<div id="redoc-container"></div>
<script src="https://cdn.jsdelivr.net/npm/redoc@next/bundles/redoc.standalone.js"></script>
<script>Redoc.init("spec.yaml", { enumSkipQuotes: true })</script>

zalesky pushed a commit that referenced this issue Oct 27, 2021
fix: Default boolean property value not rendered #1779
8609c43
@zalesky zalesky mentioned this issue Oct 27, 2021
Merged
3 tasks
zalesky added a commit that referenced this issue Oct 27, 2021
@zalesky
fix: Default boolean property value not rendered #1779 (#1781)
734080c
glosier added a commit to customerio/redoc that referenced this issue Dec 5, 2021
@glosier @stasiukanya
@RomanHotsiy @kscheirer @baijunyao @AntonKozachuk @justjavac @patrickdemers6 @kurtfurbush @andriyl @olivierbeaulieu @langhabel @hyzyla @AlexVarchuk @Leocete @Marshevskyy @waldyrious @mugx-fc @valdirmendesdev @mariusbutuc @Rkallenkoot @HCloward @adamaltman @zalesky @ivana-isadora @swapnilogale @theisenmark @kylenarocroc @gabrielsimongianotti @raghavi92 @Oprysk
Redoc update (#5)
63ead20 
* feat: add new option hideSchemaPattern (Redocly#1475)

* feat: add new option hideSchemaPattern
* chore: add hideSchemaPattern option to readme

* fix: add missed labels to elements (Redocly#1445)

* chore: upgrade prism.js

fixes Redocly#1455

* chore: Release 2.0.0-rc.48 🔖

* docs: fix `menuToggle` default value

fixes Redocly#1476

[ci skip]

* docs: fix the Rebilly link

* chore(cli): v0.10.2

* fix: fix SourceCodeWithCopy component to be non-pure

* docs: document theme options (Redocly#1438)

* chore: remove duplicate semicolons (Redocly#1504)

* fix: crash on multiple examples on parameter object

fixes Redocly#1485

* fix: pass boolean and number values as a string in highlight function (Redocly#1512)

* chore: Release 2.0.0-rc.49 🔖

* chore: update travis badge url

* fix: background-color in search results (Redocly#1531)

* fix: background-color in search results

* Delete some changes

* chore: clean up .travis.yaml

* chore(cli): upgrade redoc to rc.49

* chore(cli): v0.10.3

* fix: false-positive recursive tag case when using oneOf + allOf (Redocly#1534)

* fix: add includes polyfill

fixes Redocly#1530

* chore: Release 2.0.0-rc.50 🔖

* chore(cli): v0.10.4

* docs: remove outdated config option from readme (Redocly#1543)

* docs: minor syntax highlight fix on the README (Redocly#1549)

* feat: add disable-google-font parameter to serve command in cli (Redocly#1558)

* docs: remove extra word in grammar (Redocly#1539)

* chore: use openapi-core to bundle definition instead of json-schema-ref-parser (Redocly#1500)

* chore: use openapi-core to bundle definition instead of json-schema-ref-parser
* chore: update: root and demo webpack configs
* chore: refactor loadAndBundleSpec
* chore: update vesions: openapi-core, redoc
* chore: rolled back webpack.config(s) to previous version
* chore: changed the way of creating core Config for bundling definition
* fix: disable eslint react/display-name
* fix: update nodejs version to 12 in travis config
* chore: optimize verification: browser or server
* chore: add fetch only for browser
* Update src/utils/loadAndBundleSpec.ts
* chore: isBrowser verification improvment
Co-authored-by: Roman Hotsiy <[email protected]>

* chore: unit tests to check loading and bundling api definitions with openapi-core (Redocly#1568)

* chore: Release 2.0.0-rc.51

* chore(cli): v0.11.0

* chore(cli): v0.11.1

* fix: broken paths when budle or serve using cli (Redocly#1572)

* chore(cli): v0.11.2

* chore: update openapi-core version to fix parsing root document

* chore: Release 2.0.0-rc.52

* chore: Release 2.0.0-rc.53

* fix: added missing semicolon to styling (Redocly#1578)

* docs: update specUrl (Redocly#1577)

* chore(cli): v0.11.3

* fix: right absolute path for load and bundle definition (Redocly#1579)

* chore(cli): v0.11.4

* chore: update(cli) engine node version >=12 (Redocly#1583)

* fix: parse json theme string for standalone tag (Redocly#1492)

* fix: use operation path if operation summary/description is not provided (Redocly#1596)

resolves Redocly#1270

* feat: add basic support openApi 3.1 (Redocly#1622)

* chore: upgrade dependencies (Redocly#1634)

* chore: modernize build pipeline (Redocly#1635)

* chore: remove extra webpack resolver

* feat: merge refs oas 3.1 (Redocly#1640)

* chore: v2.0.0-rc.54

* chore(cli): upgrade redoc to rc.54

* chore(cli): v0.12.0

* chore(cli): v0.12.1

* feat: added git folder sync config

* chore: change cors url in the demo

* fix: broken linkify

fixes Redocly#1655

* fix: fix accidentally removed onLoaded

fixes Redocly#1656

* chore: fix sync workflow

* chore: v2.0.0-rc.55

* chore: fix demo slack url

* chore: changed docs sync flow

* chore: added security section into pull request template

* docs: improve the redoc cli README (Redocly#1679)

- Break some long lines
- Add some articles (the, a)
- Add a couple links for additional context
- Expand the SSR acronym
- Describe what --watch does

* feat: added localization for some labels (Redocly#1675)

* feat: add yaml highlight (Redocly#1684)

* feat: add yaml highlight

* chore: upgrade prism version

* chore(cli): v0.12.2

* chore: bumps npm-shrinkwrap (Redocly#1688)

* chore: upgrade shrinkwrap

* chore: fix dockerfile

* fix: handle empty object in security array (Redocly#1678)

* docs: add real-life usage example: BoxKnight (Redocly#1651)

* chore: add github template (Redocly#1708)

* chore: add headers to github templates (Redocly#1710)

* fix: improve openapi 3.1 (Redocly#1700)

* fix: hideLoading options in redoc standalone (Redocly#1709)

* feat: add github action to build docker images and push to ghcr.io on release (Redocly#1614)

* fix: nullable object's fields were missing (Redocly#1721)

* chore: up version for release (Redocly#1722)

* chore: add change log for v2.0.0-rc.56(Redocly#1723)

* chore(cli): v0.12.3

* docs(GSOD): Adds new quickstart doc (Redocly#1718)

* docs: fix broken links in intro (Redocly#1730)

* Removes content that is now in documentation and adds link to docs

* fix: Schema for events incorrectly omits readOnly and includes writeOnly (Redocly#1720 Redocly#1540)

* Apply suggestions from code review

* Update README.md

Co-authored-by: Adam Altman <[email protected]>

* Updates readme with about section and organizes features list

* Adds some notes to deployment

* feat: new option generatedPayloadSamplesMaxDepth (Redocly#1642)

* chore: update libs (Redocly#1707)

* fix: Redoc spelling

* Updates per reviews and adds new logo

* Minor edits

* fix: scrolling to the first item (Redocly#1753)

* Adds comparison table and OpenAPI link section

* Minor edit

* Updates image src

* fix: OpenAPI 3.1: Missing description when $ref used Redocly#1727

* fix: OpenAPI 3.1: Missing description when $ref used Redocly#1727

* fix: improve publish action scripts (Redocly#1729)

* feat: add q/kdb+ syntax highlighting (Redocly#1605)

* fix: The number of items in the array in the array is incorrect Redocly#1762 (Redocly#1763)

* fix: fix deref logic for oas3.1 (Redocly#1767)

* chore: remove commented out code (Redocly#1768)

* fix: No match scenario in search (Redocly#1667)

Co-authored-by: Alex Varchuk <[email protected]>

* chore: v2.0.0-rc.57 (Redocly#1770)

* chore: update publish cli workflow

* chore: fix publish workflow and disable s3 demo

* chore: fix publish workflow again

* chore(cli): v0.13.0 (Redocly#1771)

* chore: fix cli publish workflow

* fix: Default boolean property value not rendered Redocly#1779 (Redocly#1781)

* chore: remove broken badges from README

* [GSoD]Reorganizes the quickstart into a deployment guide and quickstart (Redocly#1749)

* Reorganizes the quickstart into a deployment guide and quickstart

* Minor edits

* fix: minor formatting issue

* fix: simplify phrasing

* Adds redirects, link to preview-docs docs, and other review edits

* remove IE polyfills as IE is no longer supported

Co-authored-by: Ivana Isadora Devcic <[email protected]>

* sync: Synced local 'docs/' with remote 'docs/redoc/'

* fix: mobile view in docker image (Redocly#1795)

* sync: Synced local 'docs/' with remote 'docs/redoc/'

* sync: Synced local 'docs/' with remote 'docs/redoc/'

* chore: run prettier & add husky pre-commit hook (Redocly#1800)

* fix: exclusiveMin/Max shows incorect range (Redocly#1799)

* fix: exclusiveMin/Max shows incorect range

* cover all number range cases & add unit tests

* add more tests

* fix maximum value

* simplify humanizeNumberRange function

* simplify exclusive checks

* Update src/utils/openapi.ts

Co-authored-by: Roman Hotsiy <[email protected]>

* update test coverage

* linting

* revert weird prettier changes

* add md files to prettier ignore

Co-authored-by: Roman Hotsiy <[email protected]>

* fix: add browser build for webpack 5 (Redocly#1796)

* chore: v2.0.0-rc.58 (Redocly#1808)

* chore(cli): v0.13.1 (Redocly#1809)

* fix this

Co-authored-by: Anna Stasiuk <[email protected]>
Co-authored-by: romanhotsiy <[email protected]>
Co-authored-by: jabba-jedi <[email protected]>
Co-authored-by: Karl Scheirer <[email protected]>
Co-authored-by: baijunyao <[email protected]>
Co-authored-by: Anton Kozachuk <[email protected]>
Co-authored-by: Oleksiy Kachynskyy <[email protected]>
Co-authored-by: 迷渡 <[email protected]>
Co-authored-by: Patrick Demers <[email protected]>
Co-authored-by: Kurt Furbush <[email protected]>
Co-authored-by: Andriy Leliv <[email protected]>
Co-authored-by: Olivier Beaulieu <[email protected]>
Co-authored-by: langhabel <[email protected]>
Co-authored-by: Yevhenii Hyzyla <[email protected]>
Co-authored-by: AlexVarchuk <[email protected]>
Co-authored-by: Ivan Shvets <[email protected]>
Co-authored-by: Ivan Shvets <[email protected]>
Co-authored-by: Roman <[email protected]>
Co-authored-by: Waldir Pimenta <[email protected]>
Co-authored-by: ILya Volchenkov <[email protected]>
Co-authored-by: Andrea Mugx <[email protected]>
Co-authored-by: Valdir Mendes <[email protected]>
Co-authored-by: Marius Butuc <[email protected]>
Co-authored-by: Roelof Kallenkoot <[email protected]>
Co-authored-by: Heather Cloward <[email protected]>
Co-authored-by: Adam Altman <[email protected]>
Co-authored-by: Andriy Zaleskyy <[email protected]>
Co-authored-by: Ivana Isadora Devcic <[email protected]>
Co-authored-by: Swapnil Ogale <[email protected]>
Co-authored-by: Adam Altman <[email protected]>
Co-authored-by: Mark Theisen <[email protected]>
Co-authored-by: Roman Marshevskyy <[email protected]>
Co-authored-by: kylenarocroc <[email protected]>
Co-authored-by: Gabriel Simon Gianotti <[email protected]>
Co-authored-by: raghavi92 <[email protected]>
Co-authored-by: redocly-bot <null>
Co-authored-by: Oprysk Vyacheslav <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zalesky zalesky

Labels
Type: Bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Default boolean property value not rendered #1779 Redocly/redoc
3 participants
@RomanHotsiy @cmeeren @zalesky