C9S/mono
3
0
Fork 0
Code Issues 2 Pull Requests 11 Actions Packages Projects 3 Releases Wiki Activity

Эпик 1 ревью: guards membership + MEDIUM/LOW #1

Merged
ant merged 1 commits from chore/review-E1-onboarding-fixes into marketplace2 2026-05-20 12:01:37 +00:00
Conversation 1 Commits 1 Files Changed 7
+46 -21
ant commented 2026-05-20 11:33:32 +00:00
Owner
Copy Link
Copy Source

Ревью Эпика 1. CRITICAL: добавлены MarketplaceMembershipGuard на category-tree/attribute/request резолверы (16 ранее unauthenticated queries). MEDIUM: organization в applicable_account_types, валидация accepted_at (ISO + не в будущем). LOW: фикс опечатки braname://TODO, устаревший warn про PR #359. Тесты: 167/169 marketplace зелёные (2 фейла в geocoder-factory — Эпик 2, фикс в отдельном PR), tsc чист. Открытые HIGH+CRITICAL фоллоуапы — в commit message.

Ревью Эпика 1. CRITICAL: добавлены MarketplaceMembershipGuard на category-tree/attribute/request резолверы (16 ранее unauthenticated queries). MEDIUM: organization в applicable_account_types, валидация accepted_at (ISO + не в будущем). LOW: фикс опечатки braname://TODO, устаревший warn про PR #359. Тесты: 167/169 marketplace зелёные (2 фейла в geocoder-factory — Эпик 2, фикс в отдельном PR), tsc чист. Открытые HIGH+CRITICAL фоллоуапы — в commit message.
ant added 1 commit 2026-05-20 11:33:33 +00:00
[598-4][@ant] fix(marketplace): ревью Эпика 1 — security/membership + MEDIUM/LOW a61c5ce076
ant started working 2026-05-20 11:35:23 +00:00
ant worked for 5 seconds 2026-05-20 11:35:28 +00:00
ant reviewed 2026-05-20 12:01:11 +00:00
ant left a comment
Author
Owner
Copy Link
Copy Source

ok

ok
components/controller/src/extensions/marketplace/application/resolvers/attribute.resolver.ts
@@ -204,4 +207,2 @@
description: 'Получить статистику по атрибутам marketplace',
})
@UseGuards(GqlJwtAuthGuard, RolesGuard)
@AuthRoles(['chairman', 'member'])
ant commented 2026-05-20 11:37:11 +00:00
Author
Owner
Copy Link
Copy Source

ты их здесь убрал потому что у тебя в MarketplaceMembershipGuard равнозначная проверка?

ты их здесь убрал потому что у тебя в MarketplaceMembershipGuard равнозначная проверка?
components/controller/src/extensions/marketplace/domain/services/request-domain.service.ts
@@ -137,3 +137,3 @@
coopname: params.coopname,
username: params.username,
// braname://TODO: добавить участок поставки / получения
// TODO: добавить участок поставки / получения
ant commented 2026-05-20 11:38:34 +00:00
Author
Owner
Copy Link
Copy Source

а почему не добавлен braname?? или позже добавшиь?

а почему не добавлен braname?? или позже добавшиь?
ant merged commit 9435942b3d into marketplace2 2026-05-20 12:01:37 +00:00
ant deleted branch chore/review-E1-onboarding-fixes 2026-05-20 12:01:42 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
ant
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
ant (Алексей Муравьев) claude valeriy
No Assignees
1 Participants
Notifications
Total Time Spent: 5 seconds
ant
5 seconds
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: C9S/mono#1
Delete Branch "chore/review-E1-onboarding-fixes"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?