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

Add disable addon feature #729

Closed
wants to merge 1 commit into from

Conversation

theguy000
Copy link

Fixes #725

Add functionality to disable and enable add-ons without uninstalling them.

  • src/common/AddonDetailsModal/AddonDetailsModal.js

    • Add "Disable" button for installed add-ons.
    • Add "Enable" button for disabled add-ons.
    • Modify modalButtons to include the new buttons.
  • src/routes/Addons/Addon/Addon.js

    • Add "Disable" button for installed add-ons.
    • Add "Enable" button for disabled add-ons.
    • Modify toggleButtonOnClick function to handle disabling and enabling add-ons.
    • Update PropTypes to include disabled.
  • src/services/Core/Core.js

    • Add method to disable an add-on.
    • Add method to enable an add-on.

Fixes Stremio#725

Add functionality to disable and enable add-ons without uninstalling them.

* **`src/common/AddonDetailsModal/AddonDetailsModal.js`**
  - Add "Disable" button for installed add-ons.
  - Add "Enable" button for disabled add-ons.
  - Modify `modalButtons` to include the new buttons.

* **`src/routes/Addons/Addon/Addon.js`**
  - Add "Disable" button for installed add-ons.
  - Add "Enable" button for disabled add-ons.
  - Modify `toggleButtonOnClick` function to handle disabling and enabling add-ons.
  - Update PropTypes to include `disabled`.

* **`src/services/Core/Core.js`**
  - Add method to disable an add-on.
  - Add method to enable an add-on.
@CLAassistant
Copy link

CLAassistant commented Nov 21, 2024

CLA assistant check
All committers have signed the CLA.

@kKaskak
Copy link
Member

kKaskak commented Nov 24, 2024

Thank you for your contribution. I want to note that there are no such actions in core, which makes this PR of no value. Will you be implementing the core side integration as well?

@kKaskak
Copy link
Member

kKaskak commented Dec 14, 2024

@theguy000 I'm waiting for your answer. If you are not planning to implement the logic for this feature, I will be closing this PR soon. 💯

@kKaskak kKaskak closed this Dec 16, 2024
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.

Allow disabling add-ons without uninstalling them.
3 participants