Opened 13 years ago
Closed 13 years ago
#3993 closed enhancement (fixed)
Supporting Message search
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | 1.6 | Priority: | normal |
Severity: | trivial | Version: | 1.5.3 |
Component: | Messages | Keywords: | has-patch commit |
Cc: | sbrajesh |
Description
Hi,
Can we support message search for user messages. It will be very beneficial for custom themes loading message via ajax and/or allowing searching(I will submit an enhancement for bp-default later).
Please have a look at the attached patch. The patch is against r5690
Attachments (3)
Change History (9)
#2
@
13 years ago
- Keywords commit added
- Milestone changed from Awaiting Review to 1.6
- Version set to 1.5.3
Love it. Can you integrate it into bp-default also? For things like this, it's difficult to make API changes without a UI to test them with.
#4
@
13 years ago
Looking forward to it, thanks! Would like to get it committed sooner than later so we can test it through the beta period.
#5
@
13 years ago
Hi jjj,
Thank you for looking into it :)
here is a patch which includes the earlier patch and also does it for bp-default. For now, I have put the search box after notices. we may change the location later. It allows ajax search of messages(we may want to clean the javascript code a little bit more as I have copied and pasted from directory search, we do not need scope/extra I guess).
oops, please do not use the bp-message-search.diff, I am adding another patch.