Skip to content

Removes fetchExtensionCompatibility in UpdateController class #460

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

laoneo
Copy link
Member

@laoneo laoneo commented May 5, 2025

User description

joomla/joomla-cms#45436


PR Type

documentation


Description

  • Added documentation for removal of fetchExtensionCompatibility method.

  • Provided migration guidance to use batchextensioncompatibility instead.

  • Updated backward incompatibility notes for UpdateController changes.


Changes walkthrough 📝

Relevant files
Documentation
removed-backward-incompatibility.md
Add migration note for UpdateController method removal     

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

  • Added section documenting removal of fetchExtensionCompatibility in
    UpdateController.
  • Provided code example for migration to batchextensioncompatibility.
  • Linked related PR and described the rationale for the change.
  • +14/-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/fetchext branch May 26, 2025 15:28
    @rdeutz rdeutz restored the remove/fetchext branch May 28, 2025 16:16
    @rdeutz rdeutz reopened this May 28, 2025
    Copy link
    Contributor

    Qodo Merge was enabled for this repository. To continue using it, please link your Git account with your Qodo account here.

    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

    Copy link
    Contributor

    Qodo Merge was enabled for this repository. To continue using it, please link your Git account with your Qodo account here.

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    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