Skip to content

AE-87 [docs] Frontend development with ESM#1547

Merged
andrewnicols merged 1 commit intomoodle:mainfrom
abgreeve:AE-87-main-2
Mar 31, 2026
Merged

AE-87 [docs] Frontend development with ESM#1547
andrewnicols merged 1 commit intomoodle:mainfrom
abgreeve:AE-87-main-2

Conversation

@abgreeve
Copy link
Copy Markdown
Contributor

This provides some guidance for frontend development in Moodle going forward with our ESM system.

Copilot AI review requested due to automatic review settings March 26, 2026 02:40
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 26, 2026

Deploy Preview for moodledevdocs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 37644c8
🔍 Latest deploy log https://app.netlify.com/projects/moodledevdocs/deploys/69cb6507510a5c0008a7ef52
😎 Deploy Preview https://deploy-preview-1547--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 project configuration.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new developer guide page describing the recommended approach for Moodle frontend development with ESM, React, and TypeScript, including templated mounting, props guidance, data-fetching patterns, and theming considerations.

Changes:

  • Introduces a new “Frontend Development” guide under docs/guides/frontend/.
  • Documents React auto-initialisation via the Mustache {{#react}} helper and the default-export contract.
  • Provides guidance on props minimisation, service-module data fetching, and theming/styling compatibility.

Comment thread docs/guides/frontend/index.md Outdated
Comment thread docs/guides/frontend/index.md Outdated
Comment thread docs/guides/frontend/index.md Outdated
Comment thread docs/guides/frontend/index.md Outdated
Comment thread docs/guides/frontend/index.md Outdated
Comment thread docs/guides/frontend/index.md Outdated
Comment thread docs/guides/frontend/index.md
Comment thread docs/guides/frontend/index.md Outdated
Comment thread docs/guides/frontend/index.md Outdated
Comment thread docs/guides/frontend/index.md
@abgreeve abgreeve force-pushed the AE-87-main-2 branch 2 times, most recently from 6030255 to f087f8b Compare March 30, 2026 07:48
@abgreeve abgreeve requested a review from Copilot March 30, 2026 07:58
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment thread docs/guides/frontend/index.md Outdated
Comment thread docs/guides/frontend/index.md Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 5 comments.

Comment thread docs/guides/frontend/index.md
Comment thread docs/guides/frontend/index.md
Comment thread docs/guides/frontend/index.md
Comment thread docs/guides/frontend/index.md
Comment thread docs/guides/frontend/index.md
Comment thread docs/guides/frontend/index.md Outdated
Comment thread docs/guides/frontend/index.md
@andrewnicols andrewnicols added this pull request to the merge queue Mar 31, 2026
Merged via the queue into moodle:main with commit 4071928 Mar 31, 2026
5 of 6 checks passed
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.

3 participants