Skip to content

Add report builder datasource to list all session attendees#219

Open
FMCorz wants to merge 1 commit intocatalyst:MOODLE_403_STABLEfrom
branchup:reportbuilder
Open

Add report builder datasource to list all session attendees#219
FMCorz wants to merge 1 commit intocatalyst:MOODLE_403_STABLEfrom
branchup:reportbuilder

Conversation

@FMCorz
Copy link

@FMCorz FMCorz commented Jun 20, 2025

Hi all,

This pull request adds support for custom reports (aka report builder) by providing a data source. The latter provides access to session attendees.

Please let me know if you have any comments.

Thank you!

@golenkovm
Copy link
Contributor

Hi @FMCorz thanks for your pull request. I will be happy to review it later this week.

Cheers,
Misha

@golenkovm
Copy link
Contributor

Hi @FMCorz just wanted to let you know that we went ahead an implemented the RB datasource and entities in #251 Please feel free to give it a test and rebase/update your PR if there are any missing bits.

Kind regards,
Misha

@FMCorz
Copy link
Author

FMCorz commented Aug 27, 2025

Hi @golenkovm ,

Thanks for letting us know. At first glance, it looks like there are significantly less entities, and less columns in those entities, than in our patch. This will probably require a large rewrite of our patch to maintain the functionality we built and use.

Were there any parts of our PR that were inadequate to being merged?

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