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

Fix the ordered list discrepancy on getting started page #665

Merged

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
2 checks passed
"description": "The price of the product",
"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