-
Notifications
You must be signed in to change notification settings - Fork 113
fix: feedByIds to return share posts from user sources #3461
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
|
…icThreshold filter Co-authored-by: omBratteng <1681525+omBratteng@users.noreply.github.com>
…t types Co-authored-by: omBratteng <1681525+omBratteng@users.noreply.github.com>
| (ctx, args, { limit, offset }, builder) => | ||
| builder.limit(limit).offset(offset), | ||
| { | ||
| removeNonPublicThresholdSquads: false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The culprit 🤦♂️
|
leaving as is, and let filter them out in briefing instead |
Changes
feedByIdswas filtering out share posts fromSourceType.Userdue to filtering out non-publich threshold sources.removeNonPublicThresholdSquads: falsesince explicit ID queries should bypass source visibility rulesExample link
https://app.daily.dev/feed-by-ids?id=088gbJeif&id=yqIOzOa7R
ENG-262
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.