Skip to content

sticker: implement proper sticker support - #303

Merged
highesttt merged 3 commits into
mainfrom
highest/plat-38144
Nov 2, 2026
Merged

highesttt merged 3 commits into
mainfrom
highest/plat-38144

Conversation

@highesttt

@highesttt highesttt commented Nov 2, 2026 •

Copy link
Copy Markdown
Collaborator

Received stickers now set BridgedSticker with the sticker ID and a link to the pack's store page, and keep the raw LINE metadata in fi.mau.line.sticker. Owned sticker packs can be listed and imported through bridgev2's image pack flow, and stickers from imported packs are sent to LINE as native stickers.

Thanks to @formeo14 (#246) for the earlier sticker work.

@indent

indent Bot commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

Issues

All clear! No issues remaining. 🎉

1 issue already resolved
  • Re-sending a received LINE sticker from a pack the user doesn't own now fails outright instead of falling back to an image send. Any resolveSticker error aborts the send, and each failure also refetches the full owned catalog. (fixed by commit 96cf595)
    Found by Indent Review Agent

CI Checks

All CI checks passed on 96cf595.

Review agents

Select any unchecked box below to run or rerun that agent.

Passed (1)
  • Indent Review Agent · No remaining bugs found.
Full results

Indent Review Agent

  • Summary: No remaining bugs found.
  • Last ran on commit: 96cf5950
  • Latest result
    {
      "summary": "No remaining bugs found.",
      "findings": []
    }

Co-authored-by: formeo14 <maiki2004@gmx.de>
Comment thread pkg/connector/handlers/sticker.go Resolved
@highesttt highesttt changed the title connector: include received sticker metadata sticker: implement proper sticker support Nov 2, 2026
@highesttt
highesttt merged commit 10e0a5f into main Nov 2, 2026
9 checks passed
@highesttt
highesttt deleted the highest/plat-38144 branch November 2, 2026 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant