Skip to content

fix: update the url for component manager - #1496

Merged
kolipakakondal merged 1 commit into
masterfrom
fix_update_component_manager_url
Aug 6, 2026
Merged

fix: update the url for component manager#1496
kolipakakondal merged 1 commit into
masterfrom
fix_update_component_manager_url

Conversation

@sigmaaa

@sigmaaa sigmaaa commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

Description

Updated the URL for the component manager because the old one is no longer valid

Fixes # (IEP-XXX)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Test Configuration:

  • ESP-IDF Version:
  • OS (Windows,Linux and macOS):

Dependent components impacted by this PR:

  • Component 1
  • Component 2

Checklist

  • PR Self Reviewed
  • Applied Code formatting
  • Added Documentation
  • Added Unit Test
  • Verified on all platforms - Windows,Linux and macOS

Summary by CodeRabbit

  • Bug Fixes
    • Updated the ESP-IDF component service endpoint to ensure component installation requests use the current service path.

@sigmaaa sigmaaa self-assigned this Aug 6, 2026
@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 618755f6-03ad-4440-8643-aadb44774ea1

📥 Commits

Reviewing files that changed from the base of the PR and between e623b3a and cb73bf7.

📒 Files selected for processing (1)
  • bundles/com.espressif.idf.ui/src/com/espressif/idf/ui/installcomponents/handler/InstallIDFComponentsHandler.java

📝 Walkthrough

Walkthrough

The components installation handler now uses https://components.espressif.com/api/components instead of the legacy API endpoint.

Changes

Components API

Layer / File(s) Summary
Update components endpoint
bundles/com.espressif.idf.ui/src/com/espressif/idf/ui/installcomponents/handler/InstallIDFComponentsHandler.java
The private API_URL constant now points to the current components service path.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: kolipakakondal, andriifilippov

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: updating the Component Manager URL.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix_update_component_manager_url

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@kolipakakondal kolipakakondal left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@sigmaaa sigmaaa added this to the v4.4.0 milestone Aug 6, 2026
@XDanielPaul

XDanielPaul commented Aug 6, 2026

Copy link
Copy Markdown
Member

Sorry folks, we did not know anyone still uses this endpoint. We've dropped it due to our analytics showing no usage and the new one was components.espressif.com/api for quite some time. I'll keep your project in mind next time

@kolipakakondal
kolipakakondal merged commit 29858ae into master Aug 6, 2026
6 checks passed
@kolipakakondal
kolipakakondal deleted the fix_update_component_manager_url branch August 6, 2026 08:00
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.

3 participants