Skip to content

Issue-390#394

Closed
wdtj wants to merge 1 commit intoreplaysMike:masterfrom
wdtj:bugs/issue-390
Closed

Issue-390#394
wdtj wants to merge 1 commit intoreplaysMike:masterfrom
wdtj:bugs/issue-390

Conversation

@wdtj
Copy link
Contributor

@wdtj wdtj commented Jun 16, 2025

DigiKey search was returning every part except the one I needed.

  1. Removed context processing which seemed to overly restrict the search to one component type. 2) Increased search limits to 50 (Digikey max)
  2. Included both exact match and fuzzy match finds in the results with the exact matches first.

DigiKey search was returning every part except the one I needed.

1) Removed context processing which seemed to overly restrict the search to one component type.
2) Increased search limits to 50 (Digikey max)
3) Included both exact match and fuzzy match finds in the results with the exact matches first.
@replaysMike replaysMike self-assigned this Jun 27, 2025
@replaysMike replaysMike added the enhancement New feature or request label Jun 27, 2025
@replaysMike replaysMike added this to the v2.6.18 milestone Jun 27, 2025
CategoryFilter = taxonomies.Any()
? taxonomies.Select(x => new FilterId() { Id = ((int)x).ToString() }).ToList()
: new List<FilterId>(),
//CategoryFilter = taxonomies.Any()
Copy link
Owner

Choose a reason for hiding this comment

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

I will include this as part of v2.6.18 - I don't know if it will have any negative effects but seems to warrant some testing

if (!response.Products.Any()) return Task.CompletedTask;

var imagesAdded = 0;
foreach (var part in response.ExactMatches)
Copy link
Owner

Choose a reason for hiding this comment

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

moving this to the new part info processor in v2.6.18

@replaysMike
Copy link
Owner

Looks good.

Closing the PR and will apply these changes against the new architecture in v.2.6.18 today

replaysMike pushed a commit that referenced this pull request Jun 27, 2025
replaysMike pushed a commit that referenced this pull request Jul 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants