romfast/gomag-vending
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(qa): ISSUE-001 — SKU prices 404 for SKUs containing slashes

Browse Source 
SKUs with forward slashes (e.g. 0V2071/250159/250158) caused a 404
on /api/mappings/{sku}/prices because path separators in the SKU
were interpreted as URL path segments. Changed to query parameter.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Claude Agent
2026-03-27 13:16:20 +00:00
parent 5a5ca63f92
commit 7e4bbabcae
3 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
api/app/routers/mappings.py
View File

@@ -146,8 +146,8 @@ async def create_batch_mapping(data: MappingBatchCreate):
return {"success": False, "error": str(e)} return {"success": False, "error": str(e)}
@router.get("/api/mappings/{sku}/prices") @router.get("/api/mappings/prices")
async def get_mapping_prices(sku: str): async def get_mapping_prices(sku: str = Query(...)):
"""Get component prices from crm_politici_pret_art for a kit SKU.""" """Get component prices from crm_politici_pret_art for a kit SKU."""
app_settings = await sqlite_service.get_app_settings() app_settings = await sqlite_service.get_app_settings()
id_pol = int(app_settings.get("id_pol") or 0) or None id_pol = int(app_settings.get("id_pol") or 0) or None

2
api/app/static/js/mappings.js
View File

@@ -176,7 +176,7 @@ async function loadKitPrices(sku, container) {
if (spinner) spinner.style.display = ''; if (spinner) spinner.style.display = '';
try { try {
const res = await fetch(`/api/mappings/${encodeURIComponent(sku)}/prices`); const res = await fetch(`/api/mappings/prices?sku=${encodeURIComponent(sku)}`);
const data = await res.json(); const data = await res.json();
if (data.error) { if (data.error) {
if (spinner) spinner.innerHTML = `<small class="text-danger">${esc(data.error)}</small>`; if (spinner) spinner.innerHTML = `<small class="text-danger">${esc(data.error)}</small>`;

2
api/app/templates/mappings.html
View File

@@ -159,5 +159,5 @@
{% endblock %} {% endblock %}
{% block scripts %} {% block scripts %}
<script src="{{ request.scope.get('root_path', '') }}/static/js/mappings.js?v=13"></script> <script src="{{ request.scope.get('root_path', '') }}/static/js/mappings.js?v=14"></script>
{% endblock %} {% endblock %}