Skip to content

[pull] main from forem:main - #406

Merged
pull[bot] merged 2 commits into
amishakov:mainfrom
forem:main
Sep 30, 2026
Merged

pull[bot] merged 2 commits into
amishakov:mainfrom
forem:main

Conversation

@pull

@pull pull Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

benhalpern and others added 2 commits September 29, 2026 16:03
…3906)

* Add markdown page body to events, separate from short description

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Exclude event body fields from API JSON; add regression specs

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
The legacy URI form spotify:user:<username>:playlist:<id> bound the
<type> capture group to the username segment, so parse_input returned
the username as the resource type. The embed rendered
open.spotify.com/embed/<username>/<id> (a broken path) and, since the
username is not a key in TYPE_HEIGHT, height="px". Capture the username
separately and bind <type> to the literal "playlist" so the URI renders
open.spotify.com/embed/playlist/<id> at the correct 380px height, the
same output as the modern spotify:playlist:<id> form.

Adds a spec asserting the rendered iframe for a legacy playlist URI.
@pull pull Bot locked and limited conversation to collaborators Sep 30, 2026
@pull pull Bot added the ⤵️ pull label Sep 30, 2026
@pull
pull Bot merged commit c4d3bb6 into amishakov:main Sep 30, 2026
2 of 4 checks passed

This branch had an error being deployed

1 failed deployment
production — c4d3bb61 Deployed Sep 30, 2026 by pull[bot] via deploy (production) #382
staging — c4d3bb61 Deployed Sep 30, 2026 by pull[bot] via deploy (staging) #382
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants