Conversation
Add a dedicated dog-specific Kokeet laaja view from the dog profile, with legacy-style trial rows, clipboard copy support, and responsive desktop/mobile layouts. The page now carries trial type and class size through the profile DTO so sijoitus can be rendered as 1 / 12, KK, or PK in the laaja table. Files: - CHANGELOG.md - apps/web/app/(public)/beagle/dogs/[dogId]/kokeet-laaja/page.tsx - apps/web/components/beagle-dog-profile/__tests__/beagle-dog-profile-page.test.ts - apps/web/components/beagle-dog-profile/__tests__/dog-profile-trials-card.test.ts - apps/web/components/beagle-dog-profile/__tests__/dog-profile-trials-laaja-page.test.ts - apps/web/components/beagle-dog-profile/beagle-dog-profile-page.tsx - apps/web/components/beagle-dog-profile/dog-profile-trials-card.tsx - apps/web/components/beagle-dog-profile/dog-profile-trials-laaja-page-container.tsx - apps/web/components/beagle-dog-profile/dog-profile-trials-laaja-page.tsx - apps/web/components/beagle-dog-profile/index.ts - apps/web/lib/i18n/messages/beagle/dogs/profile.ts - apps/web/lib/public/beagle/dogs/profile/profile-route.ts - docs/app-usage-and-features.md - docs/features/beagle-dog-profile.md - packages/contracts/dogs/profile/beagle-profile.ts - packages/db/trials/get-beagle-trials-for-dog.ts - packages/db/trials/types.ts - packages/server/dogs/profile/get-beagle-dog-profile.ts Ref BEJ-125
…nents ref bej-125
Adds a dedicated dog-trials read path for /beagle/dogs/[dogId]/kokeet-laaja so the laaja page no longer loads the full dog profile. The new flow uses an HTTP route handler, a smaller dog-trials DTO, and coverage tests for the route, hook, service, and repository. It keeps the existing profile page flow intact. Files: - apps/web/app/api/beagle/dogs/[dogId]/trials/route.ts - apps/web/app/api/beagle/dogs/[dogId]/trials/__tests__/route.test.ts - apps/web/queries/public/beagle/dogs/profile/use-beagle-dog-trials-query.ts - apps/web/queries/public/beagle/dogs/profile/dog-trials-query-keys.ts - apps/web/queries/public/beagle/dogs/profile/__tests__/use-beagle-dog-trials-query.test.ts - apps/web/components/beagle-dog-profile/dog-profile-trials-laaja-page-container.tsx - apps/web/components/beagle-dog-profile/dog-profile-trials-laaja-page.tsx - packages/server/dogs/profile/get-beagle-dog-trials.ts - packages/server/dogs/profile/__tests__/get-beagle-dog-trials.test.ts - packages/db/dogs/profile/get-beagle-dog-trials.ts - packages/db/dogs/profile/__tests__/get-beagle-dog-trials.test.ts - packages/contracts/dogs/profile/beagle-profile.ts - packages/contracts/dogs/index.ts - packages/contracts/index.ts - packages/db/dogs/index.ts - packages/db/index.ts - packages/server/dogs/index.ts - packages/server/dogs/search/service.ts - apps/web/queries/public/beagle/dogs/index.ts Ref BEJ-125
Move dog trials normalization out of packages/db so the repository layer returns raw persistence values and the server layer owns DTO shaping. Keep award formatting, judge fallback, and date/decimal conversion in the service. Files: - packages/db/dogs/profile/get-beagle-dog-trials.ts - packages/db/dogs/profile/tests/get-beagle-dog-trials.test.ts - packages/server/dogs/profile/get-beagle-dog-trials.ts - packages/server/dogs/profile/tests/get-beagle-dog-trials.test.ts Ref BEJ-125
This matches the staged change set best: - removes the duplicate dog-trials DB query - composes dog profile trials from the existing trials repo in packages/server - moves the shared wide-view formatter into apps/web/lib - updates clipboard output to reuse the same placement formatting - adds focused formatter coverage Files: - apps/web/components/beagle-dog-profile/dog-profile-trials-laaja-table.tsx - apps/web/lib/public/beagle/dogs/profile/__tests__/trials-laaja-formatters.test.ts - apps/web/lib/public/beagle/dogs/profile/trials-laaja-formatters.ts - apps/web/lib/public/beagle/trials/clipboard.ts - packages/db/dogs/index.ts - packages/db/dogs/profile/__tests__/get-beagle-dog-trials.test.ts - packages/db/dogs/profile/get-beagle-dog-trials.ts - packages/db/index.ts - packages/server/dogs/profile/__tests__/get-beagle-dog-trials.test.ts - packages/server/dogs/profile/get-beagle-dog-trials.ts Ref BEJ-125
This matches the staged change set: - adds a minimal dog identity repo - switches the dog-trials service to use it instead of the full profile graph - keeps the full profile path unchanged - updates the related tests and exports Files: - packages/db/dogs/index.ts - packages/db/dogs/profile/__tests__/get-beagle-dog-profile-identity.test.ts - packages/db/dogs/profile/get-beagle-dog-profile-identity.ts - packages/db/index.ts - packages/server/dogs/profile/__tests__/get-beagle-dog-trials.test.ts - packages/server/dogs/profile/get-beagle-dog-trials.ts Ref BEJ-125
This matches the staged change set best: - moves the shared wide-view formatter into the trial domain - updates the dog-profile table and clipboard to import from the shared trial formatter - updates the related test - includes the architecture doc clarification that supports this layout Files: - ARCHITECTURE.md - apps/web/components/beagle-dog-profile/dog-profile-trials-laaja-table.tsx - apps/web/lib/public/beagle/dogs/profile/__tests__/trials-laaja-formatters.test.ts - apps/web/lib/public/beagle/trials/clipboard.ts - apps/web/lib/public/beagle/trials/display-formatters.ts Ref BEJ-125
…c-kokeet-laaja-page
- add a laaja view action for opening all supported dog trial PDFs - render multiple trial PDFs from trial entry IDs in one public PDF page - use pdf.js canvas rendering so stale PDFs do not block the stack - expose PDF render support from dog profile trial data ref bej-126 ..
ref bej-126
…c-scrollable-pdf-protocols-page feat(dogs/profile): add AJOK laaja PDF stack
fix: some pdf might fail if they are loaded at once
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
[codex] release 0.15.0
Code Review SummaryStatus: 1 Issue Found | Recommendation: Address before merge Overview
Issue Details (click to expand)CRITICAL
Files Reviewed (43 files)
Fix these issues in Kilo Cloud Reviewed by laguna-m.1-20260312:free · Input: 535.4K · Output: 21.1K · Cached: 6.8M |
fix: add missing error handling for trial route
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
closes bej-125
closes bej-126