Skip to content

Remove Instagram + birthday-draft hotfix - #18

Open
ock666 wants to merge 1 commit into
mainfrom
feature/remove-instagram
Open

Remove Instagram + birthday-draft hotfix#18
ock666 wants to merge 1 commit into
mainfrom
feature/remove-instagram

Conversation

@ock666

@ock666 ock666 commented Aug 31, 2026

Copy link
Copy Markdown
Owner

What

Tears out the Instagram integration (it never reliably worked — the unofficial API constantly rate-limited and hung, which is exactly why the review queue's "Check now" button was timing out). Plus a small birthday-draft hotfix.

⚠️ This branch is based on feature/year-in-review-calendar (PR #17). If you merge #17 first, this PR's diff shrinks to just the Instagram + hotfix changes.

🧹 Instagram, gone

  • Deleted instagram_client.py / instagram_poll.py and the instagrapi dependency
  • Dropped the InstagramPost model and the Person.instagram_* columns/relationship (kept EventType.instagram so historical journal entries still render)
  • Removed: the review-queue "New Instagram posts" section + approve/dismiss routes, the Settings section, person form/detail fields, the privacy note, and both JSON exports
  • Stripped the poll from the daily scheduler job and from "Check now" — "Check now" now returns instantly (verified: ~0.01s vs hanging)

🎂 Birthday-draft hotfix

A container that (re)starts after the daily 8am run used to leave in-window birthdays without a draft for up to ~24h. Now birthday drafts are generated once on startup too, so new/edited birthdays always have a message ready whenever the container comes up.

What's New

Added an "Instagram, gone" note — the existing Kin Wrapped / Calendar sync / Two weeks to plan sections are untouched. APP_VERSION2026.09.3.

Tests

Full suite: only the 5 documented pre-existing failures.

Instagram never reliably worked (the unofficial API constantly rate-limited
and hung), and its poll was what made the review queue's 'check now' time out.
Removed it entirely:

- Delete instagram_client.py / instagram_poll.py and the instagrapi dependency
- Drop the InstagramPost model and Person instagram_* columns/relationship
  (kept EventType.instagram so historical entries still render)
- Remove the review-queue Instagram posts section and approve/dismiss routes,
  the Settings section, person form/detail fields, privacy note, and exports
- Strip the instagram poll from the daily scheduler job and 'check now'
- Remove instagram_username from friend-rank/contact signal + gamification

Birthday-draft hotfix: a container that (re)starts after the daily 8am run
now generates birthday drafts once on startup, so in-window birthdays always
have a message ready instead of waiting up to ~24h.

What's New: added the 'Instagram, gone' note (existing sections kept);
APP_VERSION -> 2026.09.3.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant