Skip to content

Conversation

@dzemanov
Copy link
Member

@dzemanov dzemanov commented Oct 1, 2025

Hey, I just made a Pull Request!

Updates the package disable endpoint to a PATCH. Existing packages in the configuration can be disabled without requiring the full configuration replacement.

Fixes

Fixes https://issues.redhat.com/browse/RHDHBUGS-2094

How to test

  1. Enable package not already in config
curl -X PATCH "http://localhost:7007/api/extensions/package/marketplace-plugin-demo/red-hat-developer-hub-backstage-plugin-bulk-import/configuration/disable" \
-H "Content-Type: application/json" \
-H "Authorization: Bearer $token" \
-d '{
    "disabled": false
}'
  1. Disable the same package, already in config
curl -X PATCH "http://localhost:7007/api/extensions/package/marketplace-plugin-demo/red-hat-developer-hub-backstage-plugin-bulk-import/configuration/disable" \
-H "Content-Type: application/json" \
-H "Authorization: Bearer $token" \
-d '{
    "disabled": true
}'

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or Updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)

Signed-off-by: Dominika Zemanovicova <[email protected]>
@rhdh-gh-app
Copy link

rhdh-gh-app bot commented Oct 1, 2025

Important

This PR includes changes that affect public-facing API. Please ensure you are adding/updating documentation for new features or behavior.

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-marketplace-backend workspaces/marketplace/plugins/marketplace-backend minor v0.10.0

@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 1, 2025

Copy link
Member

@debsmita1 debsmita1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

verified the changes against the UI PR #1558

/lgtm

Thank you @dzemanov !

@openshift-ci openshift-ci bot added the lgtm label Oct 3, 2025
@debsmita1 debsmita1 merged commit 36a7d6a into redhat-developer:main Oct 3, 2025
9 checks passed
@dzemanov dzemanov deleted the fix-marketplace-package-disable branch October 3, 2025 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants