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

Updated articles as per styleguide #6930

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

eshanrnh
Copy link
Contributor

@eshanrnh eshanrnh commented Mar 11, 2025

Description

  • Updated articles in the Collections section as per styleguide.
  • Provided headings for method name and changed format for displaying syntax.
  • Shortened sentences
  • Updated descriptions where required
  • Added descriptions to explain code samples where required.

Type of suggestion

  • Typo/grammar fix
  • Updated outdated content
  • New content
  • Updates related to a new version
  • Other

Product & version (if relevant)

UI Builder v15

Deadline (if relevant)

Anytime

@eshanrnh eshanrnh marked this pull request as ready for review March 12, 2025 08:45
Copy link
Contributor

@sofietoft sofietoft left a comment

Choose a reason for hiding this comment

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

I made a couple of comments regarding List View, but that was before I noticed there was also some docs related to Collection. Not really sure how to approach this to be honest 😅

Is List View not changed to Collection in UI Builder? Or are we talking about different things than in the CMS? 🤔

Also, there are a couple of style guide warnings - I believe related to the spelling of Data Type.

@eshanrnh
Copy link
Contributor Author

Thanks for the review, @sofietoft 💪 I've fixed the vale warning/errors.

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

Successfully merging this pull request may close these issues.

2 participants