Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Move lookup-related functions from RoomMemberHandler to IdentityHandler#5978

Merged
anoadragon453 merged 5 commits intodevelopfrom
anoa/lookup_to_identity_handler
Sep 27, 2019
Merged

Move lookup-related functions from RoomMemberHandler to IdentityHandler#5978
anoadragon453 merged 5 commits intodevelopfrom
anoa/lookup_to_identity_handler

Conversation

@anoadragon453
Copy link
Copy Markdown
Member

@anoadragon453 anoadragon453 commented Sep 4, 2019

Just to have all the methods that make calls to identity services in one place.

@anoadragon453
Copy link
Copy Markdown
Member Author

Removing tags as this is just a general cleanup job at this moment and not blocking the sprint.

@anoadragon453 anoadragon453 force-pushed the anoa/lookup_to_identity_handler branch from ee499bf to 163cd4f Compare September 26, 2019 15:31
@anoadragon453 anoadragon453 marked this pull request as ready for review September 26, 2019 16:21
@anoadragon453 anoadragon453 requested a review from a team September 26, 2019 16:22
@anoadragon453 anoadragon453 force-pushed the anoa/lookup_to_identity_handler branch from d64f6a0 to 9c16e00 Compare September 26, 2019 16:27
Copy link
Copy Markdown
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm taking it on trust that you are able to ctrl-x ctrl-v correctly

@anoadragon453
Copy link
Copy Markdown
Member Author

I'm taking it on trust that you are able to ctrl-x ctrl-v correctly

Goodness no sir, I used <shift>v<ctrl>ddgfo<esc>p. For you see, I use vim.

@anoadragon453 anoadragon453 merged commit 8c27bc8 into develop Sep 27, 2019
@anoadragon453 anoadragon453 deleted the anoa/lookup_to_identity_handler branch September 27, 2019 09:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants