-
Notifications
You must be signed in to change notification settings - Fork 113
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
Add user authentication and the possibility to trigger/send a message… #207
base: master
Are you sure you want to change the base?
Add user authentication and the possibility to trigger/send a message… #207
Conversation
When I call
Do you not get this too? It's odd, as the trigger() method should put POST as the argument, so I'm not sure where its going missing. |
… to a specific user
5aaeaec
to
f09845c
Compare
@benjamin-tang-pusher Yes, you are right. I found the problem and fixed it. The @request_method is unnecessary because the send_to_user is only a wrapper calling the trigger. Now, the code is fixed and working. Please let me know if you have any other problems. :) |
Whoops, I forgot about that decorator. I gave it another test and it works now. I will ask the team to review it. |
@fbenevides do you have any news? Thx. |
+1, need this |
Any news here? |
@fbenevides any updates on this PR? Would be super helpful to have 😄 |
@amad @robertoles @benjamin-tang-pusher @benw-pusher I'm tagging you because you were the last committers on this project. Do you have any intention to finish this PR? Thanks for any help you can give me. :) |
What does this PR do?
Add the authenticate_user method allowing clients to authenticate users.
Add the send_to_user_method allowing any backend to send messages to a specific user.
requirements.txt
andsetup.py
have been updatedCHANGELOG