-
Notifications
You must be signed in to change notification settings - Fork 813
Added info on usage of Composer... #6919
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
Conversation
15/umbraco-ui-builder/guides/creating-your-first-integration.md
Outdated
Show resolved
Hide resolved
15/umbraco-ui-builder/guides/creating-your-first-integration.md
Outdated
Show resolved
Hide resolved
alina-tincas
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @eshanrnh thank you for making these changes! 😊
I have few suggestions, otherwise so far it looks good 💪 🙌
15/umbraco-ui-builder/guides/creating-your-first-integration.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Alina-Magdalena Tincas <[email protected]>
Co-authored-by: sofietoft <[email protected]>
|
Thanks for the review, @alina-tincas @sofietoft 🙌 I've incorporated the suggestions 💪 |
alina-tincas
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm 💪
Description
... as that is the recommended approach.
Also, I updated a few statements here and there.
Type of suggestion
Product & version (if relevant)
UI Builder v15
Deadline (if relevant)
Anytime