Skip to content

Conversation

@enrroquez
Copy link

Hi, this is a small contribution to the Installation docs, based on this discussions in the community forums:

Thanks for all the work you do for the Digital Commons.

Hi, this is a small contribution to the Installation docs, based on this discussions in the community forums:

* https://moodle.org/mod/forum/discuss.php?d=466563
* https://moodle.org/mod/forum/discuss.php?d=466603

Thanks for all the work you do for the Digital Commons.
@netlify
Copy link

netlify bot commented Mar 13, 2025

Deploy Preview for moodledevdocs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit f2c5399
🔍 Latest deploy log https://app.netlify.com/sites/moodledevdocs/deploys/67d2b3c9f1a78f0008db4be9
😎 Deploy Preview https://deploy-preview-1294--moodledevdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@andrewnicols
Copy link
Member

Closing this issue. This is not a requirement.

@enrroquez
Copy link
Author

enrroquez commented Mar 13, 2025

Hi Andrew, I am not familiar to the process of reviewing pull requests. I interpret you did not accept my pull request, which was a suggestion to improve the documentation. Would you be so kind to tell me why?

@andrewnicols
Copy link
Member

Hi @enrroquez,

I've rejected this pull request because it is not a requirement of Moodle. There is currently a bug, which will be fixed in the next few weeks, which impacts hosts which disable pcnt_signal (MDL-84446).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants