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
Slack notification: get user by email instead of searching all users #5091
Slack notification: get user by email instead of searching all users #5091
Conversation
I have no idea if this works as I just copied the suggestion into a pr. Anyone able to test it? |
Oh I assumed it was tested before being made a PR. Maybe convert it to a draft so it does not accidentally get merged? |
Of you dismiss your review is pretty clear I think with the comments and all. |
thanks for testing, but your error seems completely unrelated to the one from the original reporter. can you open your own issue and put your details / screenshots in there? |
@valentijnscholten is the solution you have ready to go? I didn't think it was in working order yet |
@hackthesoul Can you test this so we can include it in the release today? |
No more time to test this and not many reports, moving to 2.5.0. |
9ca2472
to
7d477c5
Compare
@valentijnscholten What is your recommendation how to proceed with this PR, if we cannot test it? |
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.
Approved
fixes #4937