-
Notifications
You must be signed in to change notification settings - Fork 299
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
[docs] Migrate Plugin contribution docs #696
Conversation
✅ Deploy Preview for moodledevdocs ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
⚡️ Lighthouse report for the deploy preview of this PR
|
Hello @sarjona, would you be able to review the final migration of the plugin contribution documentation? Thank you. |
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.
Hola @vmdef!
Thanks for migrating the rest of the Plugins contribution dev pages! Well done! As you can see, I raised a few suggestions that I think will help to improve them :-)
Dear @sarjona, Thank you for your thoughtful review and valuable suggestions. I have made the following changes through new commits:
|
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.
Hola @vmdef!
Thanks for reviewing the doc and including the suggestions! As you'll see, there are only a couple of minor things to review (and then it will be ready to be merged!).
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.
Hola Víctor!
Thanks!! These pages look great and it's great they have been migrated. Well done! <3
The remaining plugin contribution documentation is being migrated.