Skip to content

[docs] Initial draft of the REST style guide - #1649

Merged
traviswindsormoodle merged 1 commit into
moodle:mainfrom
andrewnicols:restStyle
Oct 2, 2026
Merged

traviswindsormoodle merged 1 commit into
moodle:mainfrom
andrewnicols:restStyle

Conversation

@andrewnicols

@andrewnicols andrewnicols commented Jul 23, 2026 •

Copy link
Copy Markdown
Member

This is the initial draft of the REST style guide for RFC.

Copilot AI review requested due to automatic review settings July 23, 2026 06:01
@netlify

netlify Bot commented Jul 23, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for moodledevdocs ready!

Name Link
🔨 Latest commit 7ed2fe4
🔍 Latest deploy log https://app.netlify.com/projects/moodledevdocs/deploys/6abe70d8e9ea6c00084719c4
😎 Deploy Preview https://deploy-preview-1649--moodledevdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 REST documentation section to the Moodle DevDocs site, including an initial (but intended-to-be-normative) REST API style guide to standardize URI structure, naming, payload conventions, and HTTP semantics across Moodle components.

Changes:

  • Introduces a comprehensive REST API style guide covering resource design, URI conventions, JSON conventions, pagination, errors, and idempotency.
  • Adds a REST guide index page under docs/guides/rest/.

Reviewed changes

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

File Description
docs/guides/rest/style.md Adds the initial REST style guide content and examples.
docs/guides/rest/index.md Adds the REST guides section index frontmatter (and should include navigational content).
Comments suppressed due to low confidence (2)

docs/guides/rest/style.md:998

  • This TODO note also uses broken Markdown emphasis ("-*TO DO:**"). Formatting it consistently will improve readability of the draft.
-*TO DO:** this section is under revision; most of it is being removed as incorrect, so its rationale will be drafted once the replacement content is agreed.

docs/guides/rest/index.md:7

  • docs/guides/rest/index.md currently contains only frontmatter, so the rendered page will be empty. Adding a short introduction and links (e.g. to the style guide) will make the section navigable.
---

Comment thread docs/guides/rest/style.md Outdated
Comment thread docs/guides/rest/style.md Outdated
Comment thread docs/guides/rest/style.md Outdated
Comment thread docs/guides/rest/style.md
Comment thread docs/guides/rest/style.md
Comment thread docs/guides/rest/style.md
Comment thread docs/guides/rest/style.md Outdated
Comment thread docs/guides/rest/style.md
Comment thread docs/guides/rest/style.md Outdated
Comment thread docs/guides/rest/style.md Outdated
Comment thread docs/guides/rest/style.md Outdated
Comment thread docs/guides/rest/style.md Outdated
Comment thread docs/guides/rest/style.md
Comment thread docs/guides/rest/style.md Outdated
Comment thread docs/guides/rest/style.md
Comment thread docs/guides/rest/style.md

@jtse jtse left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks for writing this! A general point: Consider having fewer SHOULDs, either upgrading them to MUST or removing them. SHOULD in specs tend to make things more complicated.

Comment thread docs/guides/rest/style.md

@jtse jtse left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looking forward to seeing how this evolves and prototype implementations.

@traviswindsormoodle traviswindsormoodle left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Good to go

@traviswindsormoodle
traviswindsormoodle added this pull request to the merge queue Oct 2, 2026
Merged via the queue into moodle:main with commit 2a66405 Oct 2, 2026
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.

5 participants