[bounty] $150 fix /search
Closed this issue · 6 comments
definition of done:
- we have unit test that cover all edge cases
- /search works for all edge cases
i noticed /search broke recently (either related to UI monitoring or speaker identification features), how to reproduce:
- search with both app name and content type will ignore the content type
for example
curl ... app_name=cursor&content_type=audio
will ignore the content type
PS: i thought our tests covered all cases, or something changed, used to cover all and pass
/bounty 150
💎 $150 bounty • Screenpi.pe
Steps to solve:
- Start working: Comment
/attempt #754with your implementation plan - Submit work: Create a pull request including
/claim #754in the PR body to claim the bounty - Receive payment: 100% of the bounty is received 2-5 days post-reward. Make sure you are eligible for payouts
Thank you for contributing to mediar-ai/screenpipe!
Add a bounty • Share on socials
| Attempt | Started (GMT+0) | Solution |
|---|---|---|
| 🔴 @Yash-sudo-web | Nov 25, 2024, 7:26:46 PM | WIP |
| 🟢 @Neptune650 | #790 |
/attempt #754
Options
definition of done:
* we have unit test that cover all edge cases * /search works for all edge casesi noticed /search broke recently (either related to UI monitoring or speaker identification features), how to reproduce:
* search with both app name and content type will ignore the content typefor example
curl ... app_name=cursor&content_type=audio
will ignore the content type
PS: i thought our tests covered all cases, or something changed, used to cover all and pass
/bounty 150
@louis030195 This indeed seems to have been caused by the UI monitoring changes
I believe the culprit change is here 72bd255#diff-170633f2f9d0398cef747212489ed3dca4efe0d0f20e0813141252ecc25548ceR502-R506
Will send a patch soon
💡 @Neptune650 submitted a pull request that claims the bounty. You can visit your bounty board to reward.
🎉🎈 @Neptune650 has been awarded $150! 🎈🎊