Skip to content
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

Session report does not synchronise #513

Closed
xmontana opened this issue Aug 31, 2023 · 5 comments
Closed

Session report does not synchronise #513

xmontana opened this issue Aug 31, 2023 · 5 comments
Labels
bug Fixes problems or reduces technical debt

Comments

@xmontana
Copy link

Hello,

We have installed version v5.0.0 (2023070300) on Moodle 3.9, everything works correctly (sessions, video records...) but the session report does not appear in the Zoom activity.

The APP in Zoom is created as Server-to-Server OAuth inside the package administrator account, and has all the documented permissions.

The cron task executes but doesn't find any meeting, and we have try to force with the reports by cli:

php get_meeting_report.php --start=2023-8-20 --end=2023-8-31
Finding meetings between 2023-8-20 to 2023-8-31
Using Dashboard API
Processing 0 meetings
DONE!
@jrchamp jrchamp added the more information needed Need more information from user label Aug 31, 2023
@jrchamp
Copy link
Collaborator

jrchamp commented Aug 31, 2023

First, is there any chance that 2023-08-20 and 2023-08-31 fixes the issue? I know that the get_meeting_reports task needs an OAuth-era refresh, but I'm hoping it's not quite this broken...

@jrchamp
Copy link
Collaborator

jrchamp commented Aug 31, 2023

I have opened an initial pull request in #514 to try and address some of the issues. Please take a look and see if it helps.

@xmontana
Copy link
Author

xmontana commented Sep 1, 2023

Thank you for your help @jrchamp

I have try both date formats (2023-8-20 and 2023-08-20). Then I try again with your PR #514 upgrading the plugin to v5.1.0 2023082400 with the 2 files changes, and try again with the same result, 0 meetings and reports.

@jrchamp jrchamp added bug Fixes problems or reduces technical debt and removed more information needed Need more information from user labels Sep 19, 2023
@jrchamp
Copy link
Collaborator

jrchamp commented Sep 30, 2023

Hi @xmontana, PR #525 has been merged into the main branch as we prepare for a release. I'm hoping it fixes the issue for you. Another thing that may help: Although the report:read:admin permission is optional when using the Dashboard data, I think the Report data may be more consistent. If either of these fixes the issue, please let us know.

@xmontana
Copy link
Author

Thank you so much @jrchamp ,

After testing your PR, we could see that we had a problem with the group permissions in the scope of the Zoom marketplace APP.

Now we have it all working! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes problems or reduces technical debt
Projects
Status: Done
Development

No branches or pull requests

2 participants