-
Notifications
You must be signed in to change notification settings - Fork 305
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 main page #694
Conversation
✅ Deploy Preview for moodledevdocs ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
9251419
to
db85bc1
Compare
⚡️ Lighthouse report for the deploy preview of this PR
|
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 a lot for migrating this page! It looks great and, as you'll see, I have only a few suggestions (that you can ignore if you don't like them).
Created a folder to include all pages related to the plugins contribution process. This patch is just for the main page.
03d1b11
to
3138b08
Compare
Co-authored-by: Sara Arjona <[email protected]> Format improvement: highlight text Co-authored-by: Sara Arjona <[email protected]> Fix links Co-authored-by: Sara Arjona <[email protected]> Fix database links Co-authored-by: Sara Arjona <[email protected]> Format improvement: add info block Co-authored-by: Sara Arjona <[email protected]>
3138b08
to
b743274
Compare
Hi, @sarjona. Thank you for your review and for providing excellent suggestions. I have implemented all of them in a new commit. |
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.
Thanks for reviewing the PR and including the suggestions <3
The page looks great so I'm happy to merge it! Well done!
There is an open issue in Tracker for this migration where the location of the documentation was noted MDLSITE-7295
I created a folder to include all pages related to the plugin's contribution process. This patch is just for the main page.
I have updated the links and retained the original content to ensure that all the relevant documentation is migrated before making any changes.