Skip to content

Remove HTMLHelper::Bootstrap('framework') #396

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 2 commits into
base: main
Choose a base branch
from

Conversation

dgrammatiko
Copy link
Contributor

@dgrammatiko dgrammatiko commented Feb 24, 2025

PR Type

Documentation


Description

  • Documented the removal of HTMLHelper::Bootstrap('framework').

  • Provided a PR link and explanation for developers.


Changes walkthrough 📝

Relevant files
Documentation
removed-backward-incompatibility.md
Documented removal of `HTMLHelper::Bootstrap('framework')`

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

  • Added a section documenting the removal of
    HTMLHelper::Bootstrap('framework').
  • Included a PR link for reference.
  • Explained the reason for the removal and guidance for developers.
  • +6/-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.
  • 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
    ⚡ Recommended focus areas for review

    Typographical Error

    The word 'framwork' in the section title and description appears to be a typo and should be corrected to 'framework'.

    ### HTMLHelper::Bootstrap('framwork') has been removed
    
    - PR: https://github.com/joomla/joomla-cms/pull/44991
    - Description: The `HTMLHelper::Bootstrap('framwork')` as it was only kept for B/C purposes in the J4/5 versions. Developers should explicitly include **only** the components that are used in their layouts.

    Copy link
    Contributor

    qodo-merge-pro bot commented Feb 24, 2025

    PR Code Suggestions ✨

    Explore these optional code suggestions:

    CategorySuggestion                                                                                                                                    Impact
    General
    Fix a typo in the word "framework"
    Suggestion Impact:The suggestion corrected the typo "framwork" to "framework" in both the section title and description, improving clarity and professionalism in the documentation.

    code diff:

    -### HTMLHelper::Bootstrap('framwork') has been removed
    +### HTMLHelper::Bootstrap('framework') has been removed
     
     - PR: https://github.com/joomla/joomla-cms/pull/44991
    -- Description: The `HTMLHelper::Bootstrap('framwork')` as it was only kept for B/C purposes in the J4/5 versions. Developers should explicitly include **only** the components that are used in their layouts.
    +- Description: The `HTMLHelper::Bootstrap('framework')` as it was only kept for B/C purposes in the J4/5 versions. Developers should explicitly include **only** the components that are used in their layouts.

    Correct the typo in the section title and description where "framwork" is
    misspelled. It should be "framework" to ensure clarity and professionalism in
    the documentation.

    migrations/54-60/removed-backward-incompatibility.md [171-174]

    -### HTMLHelper::Bootstrap('framwork') has been removed
    +### HTMLHelper::Bootstrap('framework') has been removed
     
     - PR: https://github.com/joomla/joomla-cms/pull/44991
    -- Description: The `HTMLHelper::Bootstrap('framwork')` as it was only kept for B/C purposes in the J4/5 versions. Developers should explicitly include **only** the components that are used in their layouts.
    +- Description: The `HTMLHelper::Bootstrap('framework')` as it was only kept for B/C purposes in the J4/5 versions. Developers should explicitly include **only** the components that are used in their layouts.

    [Suggestion has been applied]

    Suggestion importance[1-10]: 9

    __

    Why: The suggestion addresses a clear typo in the documentation, correcting "framwork" to "framework". This improves the clarity and professionalism of the documentation, which is crucial for developer guidance.

    High
    • Update

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    1 participant