wasrusgen/zov-tech
1
0
Fork 0
mirror of https://github.com/wasrusgen/zov-tech.git synced 2026-06-03 18:04:47 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

fix: права доступа по ролям — сборщик/клиент

Browse Source 
- measurement_detail: любой мастер (measurer+assembler) видит фото замера
- assembly_list: клиент видит все свои сборки (по client_tg_id)
- assembly_detail: клиент видит детальную карточку сборки

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
wasrusgen 2026-05-18 11:59:16 +03:00
parent 4386779ea6
commit 64292cef8e
2 changed files with 14 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
agents/ROLES.md
View File

@ -103,8 +103,8 @@
--- ---
## Нерешённые вопросы (требуют решения Руслана) ## Решения (зафиксированы 2026-05-18)
- [ ] Нужен ли сборщику доступ к фото замера (сейчас только логистика)? - [x] Сборщик видит фото замера + логистику ✅
- [ ] Клиент видит историю всех своих сборок или только текущую? - [x] Клиент видит все свои сборки (полная история) ✅
- [ ] Замерщик получает уведомление в боте при назначении заявки? - [x] Замерщик получает уведомление в боте при назначении заявки ✅

16
backend-py/app/main.py
View File

@ -2315,7 +2315,8 @@ def _handle_assembly_list(body: dict[str, Any]) -> dict[str, Any]:
is_manager = sheets.has_role(user, "manager") is_manager = sheets.has_role(user, "manager")
is_master = sheets.is_master(user) is_master = sheets.is_master(user)
if not is_manager and not is_master: is_client = sheets.has_role(user, "client")
if not is_manager and not is_master and not is_client:
return {"error": "forbidden"} return {"error": "forbidden"}
_ensure_assemblies_sheet() _ensure_assemblies_sheet()
@ -2342,6 +2343,8 @@ def _handle_assembly_list(body: dict[str, Any]) -> dict[str, Any]:
visible = True visible = True
elif not row.get("assigned_to_tg_id") and row.get("status") in ("created", "scheduled"): elif not row.get("assigned_to_tg_id") and row.get("status") in ("created", "scheduled"):
visible = True visible = True
if is_client and str(row.get("client_tg_id")) == str(tg_id):
visible = True
if not visible: if not visible:
continue continue
out.append({ out.append({
@ -2386,9 +2389,10 @@ def _handle_assembly_detail(body: dict[str, Any]) -> dict[str, Any]:
if not row: if not row:
return {"error": "assembly_not_found"} return {"error": "assembly_not_found"}
# Право: менеджер-владелец, назначенный мастер, неназначенная сборка (status=created) # Право: менеджер-владелец, назначенный мастер, клиент-владелец, неназначенная сборка
is_owner = str(row.get("manager_tg_id")) == str(tg_id) or \ is_owner = str(row.get("manager_tg_id")) == str(tg_id) or \
str(row.get("assigned_to_tg_id")) == str(tg_id) str(row.get("assigned_to_tg_id")) == str(tg_id) or \
str(row.get("client_tg_id")) == str(tg_id)
is_open_slot = (not row.get("assigned_to_tg_id")) and row.get("status") in ("created", "scheduled") is_open_slot = (not row.get("assigned_to_tg_id")) and row.get("status") in ("created", "scheduled")
if not is_owner and not is_open_slot: if not is_owner and not is_open_slot:
return {"error": "forbidden"} return {"error": "forbidden"}
@ -2970,11 +2974,11 @@ def _handle_measurement_detail(body: dict[str, Any]) -> dict[str, Any]:
if not row: if not row:
return {"error": "measurement_not_found"} return {"error": "measurement_not_found"}
# Доступ: владелец-менеджер, назначенный мастер (замер/сборка), клиент-владелец # Доступ: владелец-менеджер, любой мастер (замерщик или сборщик), клиент-владелец
is_owner_manager = sheets.has_role(user, "manager") and str(row.get("manager_tg_id", "")) == str(tg_id) is_owner_manager = sheets.has_role(user, "manager") and str(row.get("manager_tg_id", "")) == str(tg_id)
is_assigned_master = sheets.is_master(user) and str(row.get("assigned_to_tg_id", "")) == str(tg_id) is_master = sheets.is_master(user) # measurer или assembler — оба видят фото замера
is_client = str(row.get("client_tg_id", "")) == str(tg_id) is_client = str(row.get("client_tg_id", "")) == str(tg_id)
if not (is_owner_manager or is_assigned_master or is_client): if not (is_owner_manager or is_master or is_client):
return {"error": "forbidden"} return {"error": "forbidden"}
def _safe_json(s: str) -> Any: def _safe_json(s: str) -> Any: