Skip to content

Conversation

sofietoft
Copy link
Contributor

Description

An attempt to clean up the Common Pitfalls article.
I've made it comply with out styleguide.

Type of suggestion

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

Copy link
Contributor

@eshanrnh eshanrnh left a comment

Choose a reason for hiding this comment

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

Looks good. However, when I preview the article there's too much of text in it 🙈. I know restructuring is not the scope of this PR so it's completely fine to merge it in 💪

@sofietoft
Copy link
Contributor Author

Thanks @eshanrnh !

Yeah, it's a long article with a lot of text.
We could try using expendables? 🤔

@eshanrnh
Copy link
Contributor

eshanrnh commented Sep 3, 2024

Thanks @eshanrnh !

Yeah, it's a long article with a lot of text. We could try using expendables? 🤔

I think that might help.

@sofietoft
Copy link
Contributor Author

@eshanrnh - I'll merge this one, and play around with some expandable on GitBook.

@sofietoft sofietoft merged commit fe20c19 into main Sep 3, 2024
@sofietoft sofietoft deleted the CommonPitfalls branch September 20, 2024 10:33
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