Repository navigation
fix(feed): accept every LinkedIn post link and catch "dev weekend" typos - #86
Merged
Merged
Conversation
- The submit form and admin import now accept lnkd.in short links, ugcPost and share links, and /feed/update/ links. The activity ID /feed needs is looked up from the post's public embed. Before, only links with "activity-<id>" were accepted, so links from LinkedIn's share button were rejected. - A post counts as mentioning Dev Weekends for any spelling: "Dev Weekends", "dev weekend", "DevWeekend", "dev-weekends", "dev_weekends". Search results use the same match, so fewer real posts are missed. - Admin import is capped at 300 links per paste. - A resubmitted post that was already reviewed and hidden now says so. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
activity-<id>worked. Links from LinkedIn's share button arelnkd.inshort links or containugcPost-<id>, so they failed with "Please paste a public LinkedIn post link".What changes
https://lnkd.in/...short links (followed only on lnkd.in, at most 3 redirects)/posts/...-ugcPost-<id>and/posts/...-share-<id>. The activity ID is read from the post's public embed./feed/update/urn:li:activity|ugcPost|share:<id>?utm_source=shareare dropped.Testing
7458002700706672640. Non-LinkedIn links, profile links,http://links and broken short links are rejected.ugcPostlink and a junk link adds the post once and reports the junk link as invalid.tscandnext lintpass.🤖 Generated with Claude Code