Merge: promote dev to prod
This commit is contained in:
commit
32bd2cdefb
3 changed files with 26 additions and 3 deletions
2
VERSION
2
VERSION
|
|
@ -1 +1 @@
|
||||||
0.11.0
|
0.11.1
|
||||||
|
|
|
||||||
|
|
@ -313,7 +313,17 @@ def subscribe(
|
||||||
if channel.details_synced_at is None:
|
if channel.details_synced_at is None:
|
||||||
apply_channel_details(db, yt.get_channels([channel_id]))
|
apply_channel_details(db, yt.get_channels([channel_id]))
|
||||||
except YouTubeError as exc:
|
except YouTubeError as exc:
|
||||||
raise HTTPException(status_code=502, detail=f"YouTube subscribe failed: {exc}")
|
# Already following on YouTube but not recorded here (a desync — e.g. a stale import
|
||||||
|
# dropped the local row, so it resurfaced in discovery). That's not a failure: the end
|
||||||
|
# state the user wanted already holds, so record it locally and move on. The next
|
||||||
|
# subscription resync fills in the resource id. Any other YouTube error is a real
|
||||||
|
# problem — surface it with a clear message (422, not a 502 that the client would
|
||||||
|
# mistake for a transient "connection lost").
|
||||||
|
msg = str(exc).lower()
|
||||||
|
if "already exists" in msg or "duplicate" in msg:
|
||||||
|
yt_sub_id = ""
|
||||||
|
else:
|
||||||
|
raise HTTPException(status_code=422, detail=f"YouTube couldn't subscribe you: {exc}")
|
||||||
db.add(
|
db.add(
|
||||||
Subscription(
|
Subscription(
|
||||||
user_id=user.id,
|
user_id=user.id,
|
||||||
|
|
@ -349,7 +359,9 @@ def unsubscribe(
|
||||||
with quota.attribute(user.id, "unsubscribe"), YouTubeClient(db, user) as yt:
|
with quota.attribute(user.id, "unsubscribe"), YouTubeClient(db, user) as yt:
|
||||||
yt.delete_subscription(sub.yt_subscription_id)
|
yt.delete_subscription(sub.yt_subscription_id)
|
||||||
except YouTubeError as exc:
|
except YouTubeError as exc:
|
||||||
raise HTTPException(status_code=502, detail=f"YouTube unsubscribe failed: {exc}")
|
# 422 (a real, explainable error → speaking modal), not 502 which the client treats as
|
||||||
|
# a transient "connection lost" gateway blip.
|
||||||
|
raise HTTPException(status_code=422, detail=f"YouTube couldn't unsubscribe you: {exc}")
|
||||||
db.delete(sub)
|
db.delete(sub)
|
||||||
db.commit()
|
db.commit()
|
||||||
return {"unsubscribed": channel_id}
|
return {"unsubscribed": channel_id}
|
||||||
|
|
|
||||||
|
|
@ -14,6 +14,17 @@ export interface ReleaseEntry {
|
||||||
}
|
}
|
||||||
|
|
||||||
export const RELEASE_NOTES: ReleaseEntry[] = [
|
export const RELEASE_NOTES: ReleaseEntry[] = [
|
||||||
|
{
|
||||||
|
version: "0.11.1",
|
||||||
|
date: "2026-06-19",
|
||||||
|
summary: "A fix for subscribing from the discovery list.",
|
||||||
|
fixes: [
|
||||||
|
"Subscribing to a channel you already follow on YouTube now simply records it here instead of showing a misleading “connection lost” error — which also clears the case where such a channel kept reappearing under “Discover from playlists”.",
|
||||||
|
],
|
||||||
|
chores: [
|
||||||
|
"YouTube-side action failures now surface as a clear message instead of being mistaken for a transient connection drop.",
|
||||||
|
],
|
||||||
|
},
|
||||||
{
|
{
|
||||||
version: "0.11.0",
|
version: "0.11.0",
|
||||||
date: "2026-06-19",
|
date: "2026-06-19",
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue