Skip to content

Fix optional plugin download logic - #893

Merged
tieneupin merged 5 commits into
mainfrom
fix-plugin-download
Sep 22, 2026
Merged

tieneupin merged 5 commits into
mainfrom
fix-plugin-download

Conversation

@tieneupin

Copy link
Copy Markdown
Contributor

The get_plugin_wheel API endpoint previously did not generate the file with the correct name. This was due to a missing f-string in the "Content-Disposition" value in the header.

Additionally, a new API endpoint was added that returns a HTML page showing the plugins available for a given instrument.

Tests have also been added for these two functions as part of ongoing efforts to improve test coverage for this code base.

@tieneupin tieneupin self-assigned this Sep 21, 2026
@tieneupin tieneupin added bug Something isn't working enhancement New feature or request server Relates to the server component labels Sep 21, 2026
@codecov

codecov Bot commented Sep 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 55.05%. Comparing base (e74297e) to head (7e87a6c).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #893      +/-   ##
==========================================
+ Coverage   54.99%   55.05%   +0.06%     
==========================================
  Files         103      103              
  Lines       11523    11528       +5     
  Branches     1531     1531              
==========================================
+ Hits         6337     6347      +10     
+ Misses       4843     4838       -5     
  Partials      343      343              
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tieneupin
tieneupin merged commit 953cd41 into main Sep 22, 2026
14 checks passed
@tieneupin
tieneupin deleted the fix-plugin-download branch September 22, 2026 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request server Relates to the server component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants