feat: include animated_url in tier-up response for T3/T4 #201

New Issue

cal · 2026-04-08T00:23:29Z

cal commented

2026-04-08 00:23:29 +00:00

Problem

When a tier-up to T3 or T4 occurs, the evaluate-game endpoint returns variant_created in the tier_ups list. The Discord bot uses this to show a notification, but there is no animated_url field in the response.

The bot would need to separately construct or call the /animated endpoint to get an APNG URL for the notification embed.

Fix

Add an animated_url field to the tier-up response for T3/T4 tier-ups. This could be a pre-constructed URL to the animated endpoint, or an S3 URL if database#198 (APNG S3 pipeline) is implemented.

Relationship

Companion to discord#144 (tier-up notification should include card image). That issue covers the bot-side embed; this covers the API-side data.

## Problem When a tier-up to T3 or T4 occurs, the `evaluate-game` endpoint returns `variant_created` in the tier_ups list. The Discord bot uses this to show a notification, but there is no `animated_url` field in the response. The bot would need to separately construct or call the `/animated` endpoint to get an APNG URL for the notification embed. ## Fix Add an `animated_url` field to the tier-up response for T3/T4 tier-ups. This could be a pre-constructed URL to the animated endpoint, or an S3 URL if database#198 (APNG S3 pipeline) is implemented. ## Relationship Companion to discord#144 (tier-up notification should include card image). That issue covers the bot-side embed; this covers the API-side data.

Claude added the

ai-working

label 2026-04-08 09:31:15 +00:00

cal referenced this issue from a commit

2026-04-08 09:36:02 +00:00

feat: include animated_url in tier-up response for T3/T4 (#201)

Claude referenced a pull request that will close this issue

2026-04-08 09:36:13 +00:00

feat: include animated_url in tier-up response for T3/T4 (#201) #208

Claude added

ai-pr-opened

and removed

ai-working

labels 2026-04-08 09:36:19 +00:00

Claude commented

2026-04-08 09:36:22 +00:00

PR #208 opened: #208

Added animated_url to T3/T4 tier-up entries in evaluate-game. The URL is constructed from API_BASE_URL env var + /api/v2/players/{player_id}/{card_type}card/{date}/{variant}/animated. Set API_BASE_URL=https://pd.manticorum.com:815 in the container env to get absolute URLs for Discord embeds. T0–T2 tier-ups are unaffected.

PR #208 opened: https://git.manticorum.com/cal/paper-dynasty-database/pulls/208 Added `animated_url` to T3/T4 tier-up entries in `evaluate-game`. The URL is constructed from `API_BASE_URL` env var + `/api/v2/players/{player_id}/{card_type}card/{date}/{variant}/animated`. Set `API_BASE_URL=https://pd.manticorum.com:815` in the container env to get absolute URLs for Discord embeds. T0–T2 tier-ups are unaffected.

cal closed this issue

2026-04-08 10:25:54 +00:00

cal referenced this issue from a commit

2026-04-08 10:25:55 +00:00

Merge pull request 'feat: include animated_url in tier-up response for T3/T4 (#201)' (#208) from issue/201-feat-include-animated-url-in-tier-up-response-for into main

Sign in to join this conversation.

No Milestone

No project

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cal/paper-dynasty-database#201