Skip to content

Conversation

TamaroWalter
Copy link
Contributor

@TamaroWalter TamaroWalter commented Jul 31, 2025

Note: Please fill out all required sections and remove irrelevant ones.

🔀 Purpose of this PR:

  • Fixes a bug
  • Updates for a new Moodle version
  • Adds a new feature of functionality
  • Improves or enhances existing features
  • Refactoring: restructures code for better performance or maintainability
  • Testing: add missing or improve existing tests
  • Miscellaneous: code cleaning (without functional changes), documentation, configuration, ...

📝 Description:

Hotfix for Moodle 5.0.

  • right rendering of pictures that are added in the post
  • replacing help icon with help icon from moodle core
  • deletion of first post of discussion works right now
  • new test for post deletion

📋 Checklist

Please confirm the following (check all that apply):

  • I have phpunit and/or behat tests that cover my changes or additions.
  • Code passes the code checker without errors and warnings.
  • Code passes the moodle-ci/cd pipeline on all supported Moodle versions or the ones the plugin supports.
  • Code does not have var_dump() or var_export or any other debugging statements (or commented out code) that
    should not appear on the productive branch.
  • Code only uses language strings instead of hard-coded strings.
  • If it is a Moodle update PR: I read the release notes, updated the version.php and the CHANGES.md.
    I ran all tests thoroughly checking for errors. I checked if bootstrap had any changes/deprecations that require
    changes in the plugins UI.

@TamaroWalter TamaroWalter self-assigned this Jul 31, 2025
@TamaroWalter TamaroWalter merged commit 990bfe5 into main Aug 1, 2025
52 checks passed
@TamaroWalter TamaroWalter deleted the hotfix/M500 branch August 1, 2025 09:27
lucaboesch pushed a commit to lucaboesch/moodle-mod_moodleoverflow that referenced this pull request Aug 3, 2025
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.

1 participant