Skip to content

[MDL-83887] Update information for mod/pluginname/index.php #1408

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sarjona
Copy link
Member

@sarjona sarjona commented Jul 24, 2025

The content of the modules index.php has been updated, to reuse the information displayed in the Activities overview :-)

@Copilot Copilot AI review requested due to automatic review settings July 24, 2025 14:40
Copy link

netlify bot commented Jul 24, 2025

Deploy Preview for moodledevdocs ready!

Name Link
🔨 Latest commit 43d00a6
🔍 Latest deploy log https://app.netlify.com/projects/moodledevdocs/deploys/688245f976fc18000867ce26
😎 Deploy Preview https://deploy-preview-1408--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
Contributor

@Copilot 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

This PR updates the documentation for module index.php files to use a new centralized approach for displaying activity overviews. Instead of implementing custom logic to fetch and display activity instances, modules now redirect to a unified Activities page using the activityoverviewbase class.

Key Changes:

  • Replaced manual activity listing logic with a single redirect call
  • Updated documentation to reference the new activityoverviewbase class
  • Simplified the implementation from ~15 lines to 2 lines of functional code

Reviewed Changes

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

File Description
docs/apis/plugintypes/mod/_files/index-php.tsx Updated example code to use activityoverviewbase::redirect_to_overview_page() instead of manual activity fetching
docs/apis/plugintypes/mod/_files/index-php.mdx Added documentation explaining the new activityoverviewbase class usage

Copy link
Collaborator

@aanabit aanabit left a comment

Choose a reason for hiding this comment

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

The changes seems OK to me. I will let the approval to the integrator.

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