Skip to content

Conversation

@eshanrnh
Copy link
Contributor

@eshanrnh eshanrnh commented Feb 5, 2025

Description

  • Restructured the Getting Started and Installation section to merge the articles into one section..
  • Renamed overview to requirements as that is what the article is about.
  • Added redirects.
  • Updated the Creating your first integration guide based on tests performed.
  • Moved images to respective folder structures.
  • Took new screenshots where required.
  • Updated old naming convention to new (for example: list view to collection, content app to workspace views and so on)

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)

When should the content be published?

@eshanrnh eshanrnh marked this pull request as ready for review February 6, 2025 11:15
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.

Great work on the initial restructure here 💪

I've made a couple of suggestions and comments throughout the articles - mostly related to content.
Let me know if something doesn't make sense.

@eshanrnh
Copy link
Contributor Author

Thanks for the review, @sofietoft 💪 I've incorporated the comments and updated the images to remove the red border.

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.

Looks great 💪
I like that article that was split into headings instead - looks cleaner in a way!

I've added just a single comment more 🙈

@eshanrnh eshanrnh merged commit 28d1c62 into main Feb 10, 2025
21 checks passed
@eshanrnh eshanrnh deleted the ui-builder branch February 10, 2025 10:31
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.

3 participants