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

Update & Upgrade #7

Merged
merged 14 commits into from
Jan 16, 2025
Merged

Update & Upgrade #7

merged 14 commits into from
Jan 16, 2025

Conversation

ferishili
Copy link
Contributor

This PR fixes #6

Changes

  • new moodle coding styles! e.g. array() to [], lang sorting, etc.
  • core namespace changes: core_component to \core\component
  • use LTI credentials from Opencast API tool plugin!
  • Use the opencast API library instead of pure curl calls if necessary!
  • Support for OC 16 more in the area of either eliminating the use of search endpoint or making it (backward) compatible.
  • Upgrade behat test!
  • Use the latest moodle ci workflows!

New Setting

  • The setting related to LTI consumer key and secret are now removed from the plugin setting and has been replaced by a simple checkbox, which makes sure to Use LTI Authentication!
    The reason is to bring all credentials to one place (tool_opencast plugin) as it has been decided by the community. So we just use the LTI Credentials from Opencast API from now on!

Screenshot 2025-01-16 at 12 32 38

@ferishili ferishili added the enhancement New feature or request label Jan 16, 2025
@ferishili ferishili self-assigned this Jan 16, 2025
@bluetom bluetom self-requested a review January 16, 2025 12:24
@bluetom bluetom merged commit d9182e4 into master Jan 16, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Plugin Upgrade
2 participants