Skip to content

Conversation

@mkne
Copy link
Contributor

@mkne mkne commented Dec 11, 2025

This pull request implements the Buerklin API and as Information Provider that also implements the BatchInfoProviderInterface.
Bürklin also offers an API. (Apply here: https://www.buerklin.com/de/services/eprocurement/)
Authorization works via Bearer Token that is requested via a POST-action to https://www.buerklin.com/authorizationserver/oauth/token/ and the following parameters:
client_id:cws-client
client_secret:{{PROVIDER_BUERKLIN_SECRET}}
grant_type:password
username:{{PROVIDER_BUERKLIN_USERNAME}}
password:{{PROVIDER_BUERKLIN_PASSWORD}}

There are only two GET-actions available:

Product search (example with the part number 66S447): https://www.buerklin.com/buerklinws/v2/buerklin/products/66S447/?curr=EUR&language=de
Free text search (example with the query string 'Resistor'): https://www.buerklin.com/buerklinws/v2/buerklin/products/search/?curr=EUR&language=en&pageSize=50&currentPage=0&query=Resistor&sort=relevance
I would set the parameters curr and language via PROVIDER_BUERKLIN_CURRENCY and PROVIDER_BUERKLIN_LANGUAGE.

Bürklin supplies a postman collection which I used to test the API and reverse engineer.
For reference: #868

mkne and others added 30 commits November 5, 2024 09:14
Use Autowire to use values of environment variables
Remove unwanted Code from LCSC-Provider
Map json response to DTO variables
Ensure array keys exist before accessing them
Optimize API calls to prevent unnecessary requests
Improve error handling for better debugging
Enhance readability and maintainability of functions
Change Order of Capabilities
# Conflicts:
#	.docker/symfony.conf
#	VERSION
@codecov
Copy link

codecov bot commented Dec 11, 2025

Codecov Report

❌ Patch coverage is 66.99346% with 101 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.41%. Comparing base (98b8c5b) to head (c28c9b1).

Files with missing lines Patch % Lines
.../InfoProviderSystem/Providers/BuerklinProvider.php 66.99% 101 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1151      +/-   ##
============================================
+ Coverage     58.31%   58.41%   +0.10%     
- Complexity     7294     7384      +90     
============================================
  Files           580      581       +1     
  Lines         23207    23513     +306     
============================================
+ Hits          13534    13736     +202     
- Misses         9673     9777     +104     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

mkne and others added 13 commits December 16, 2025 20:57
…9c8c4470476a70b6f398e3d545550#r2622249199

Revert "Change order of capabilities in LCSCProvider.php"

This reverts commit dfd6f33.
…9c8c4470476a70b6f398e3d545550#r2622249861

Revert "Change order of capabilities in PollinProvider.php"

This reverts commit fc2e726.
…AuthTokenManager

Revert "add use OAuthTokenManager and create instance in constructor"This reverts commit 2a1e7c9.Revert "Add missing ) to retrieveROPCToken"This reverts commit 8df5cfc.
Revert "Implement retrieveROPCToken as proposed in Part-DB#1151 (comment)"
This reverts commit 66cc732.
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.

2 participants