Skip to content

Conversation

@laoneo
Copy link
Member

@laoneo laoneo commented May 5, 2025

User description

joomla/joomla-cms#45434


PR Type

documentation


Description

  • Document removal of getLogContentTypeParams from ActionlogsHelper

  • Provide migration instructions for affected code

  • Update backward incompatibility notes for developers


Changes walkthrough 📝

Relevant files
Documentation
removed-backward-incompatibility.md
Add documentation for removed ActionlogsHelper method       

migrations/54-60/removed-backward-incompatibility.md

  • Added section documenting removal of getLogContentTypeParams from
    ActionlogsHelper
  • Provided code migration example from old to new usage
  • Linked relevant PR and file for reference
  • +15/-0   

    Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
  • @HLeithner HLeithner added 6.0 PR for Joomla 6.0 Waiting for Joomla reference PR This pr is ready to merge and can be merged when the corresponding Joomla cms pr is merged. labels May 5, 2025
    @laoneo laoneo closed this May 26, 2025
    @laoneo laoneo deleted the remove/actionslogsfunction branch May 26, 2025 15:28
    @HLeithner HLeithner restored the remove/actionslogsfunction branch May 26, 2025 16:16
    @HLeithner HLeithner reopened this May 26, 2025
    @qodo-code-review
    Copy link
    Contributor

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    @qodo-code-review
    Copy link
    Contributor

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @HLeithner HLeithner merged commit b5cce0a into main May 26, 2025
    2 of 3 checks passed
    @HLeithner
    Copy link
    Member

    thx

    @HLeithner HLeithner deleted the remove/actionslogsfunction branch May 26, 2025 16:18
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Labels

    6.0 PR for Joomla 6.0 Review effort 1/5 Waiting for Joomla reference PR This pr is ready to merge and can be merged when the corresponding Joomla cms pr is merged.

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    3 participants