Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add SearchInput to users and rooms #46

Closed
wants to merge 3 commits into from
Closed

Add SearchInput to users and rooms #46

wants to merge 3 commits into from

Conversation

dklimpel
Copy link
Contributor

Add filter to search in user_id and room name.

@dklimpel
Add SearchInput to users and rooms
a9477c6 
Add filter to search in `user_id` and room name.
@dklimpel dklimpel mentioned this pull request May 26, 2020
Closed
@sm2x
Copy link

sm2x commented Jun 3, 2020 •
edited
Loading

Ubuntu 18.04
Matrix Synpase 1.14.0
Nginx 1.14.0
node v12.18.0
yarn 1.22.4

Google Chrome Version 83.0.4103.61
I get "Cannot read property 'users' of undefined" as soon as I cleared the search input field

@dklimpel
Copy link
Contributor Author

dklimpel commented Jun 3, 2020

I get "Cannot read property 'users' of undefined" as soon as I cleared the search input field

I cannot reproduce this neither with FF 76/77 nor with Chrome 83.

@dklimpel dklimpel closed this Jun 3, 2020
@dklimpel dklimpel reopened this Jun 3, 2020
@awesome-manuel
Copy link
Member

Please wait for matrix-org/synapse#7377

@CybotTM
Copy link

CybotTM commented Oct 13, 2020

Please wait for matrix-org/synapse#7377

seems to be merged.

@dklimpel dklimpel mentioned this pull request Nov 11, 2020
Merged
@dklimpel
Copy link
Contributor Author

Replaced by #83

@dklimpel dklimpel closed this Nov 11, 2020
@dklimpel dklimpel deleted the search_filter branch November 12, 2020 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dklimpel @sm2x @awesome-manuel @CybotTM