Skip to content

Conversation

bhavukkalra
Copy link
Contributor

@bhavukkalra bhavukkalra commented Apr 18, 2024

What kind of change does this PR introduce?
Fixes the ordered list Discrepancy observed on the getting-started-step-by-step as reported in the issue #653

Issue Number:

Screenshots of fix
image

image

Summary
It was a markdown spacing issue. Found the fix here markdown-continue-numbered-list and applied.

Note - This might need to be applied to other pages as well which has this problem.

Does this PR introduce a breaking change?
No

CC - @benjagm @gregsdennis @jviotti

@bhavukkalra bhavukkalra requested a review from a team as a code owner April 18, 2024 19:09
Copy link
Collaborator

@benjagm benjagm left a comment

Choose a reason for hiding this comment

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

It worked. Thanks a lot!!

@gregsdennis gregsdennis mentioned this pull request Apr 23, 2024
@benjagm benjagm merged commit 9d52978 into json-schema-org:main Apr 23, 2024
"type": "number"
}
}
```

Choose a reason for hiding this comment

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

this one missed indentation

Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks for finding this @notEthan !!

@bhavukkalra can you create a new PR to fix this? I merged it too fast.

"dimensions": {
"type": "object",
}
```

Choose a reason for hiding this comment

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

here too

Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks for finding this @notEthan !!

@bhavukkalra can you create a new PR to fix this? I merged it too fast.

}
}
}
```

Choose a reason for hiding this comment

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

here too

Copy link
Collaborator

Choose a reason for hiding this comment

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

Huge thanks for finding this!

Bhavuk can you create a new PR to fix this? I merged it too fast.

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.

[📝 Docs]: Fix documentation anomalies [UI/UX change]
4 participants