-
Notifications
You must be signed in to change notification settings - Fork 574
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
Feedback feature with screenshot and browser information #953
base: master
Are you sure you want to change the base?
Conversation
Could you please take a look at this. It will work fine in web browsers. Guys, |
Hi @everyone, Please don't merge this. This needs some style changes and will be integrated soon. I know it won't happen like that. I'm just saying this so that this won't happen. |
@jinojossy93 Awesome work Jino! |
Hi @biswaz , Currently, when feedbacks are saved, an error in the redis_worker is happening. This happens since I've integrated the automated message sending using Redis queue. But I'm not able to run that in my dev environment. I've used the command that you provided
It's throwing an error
Please suggest if I missed something for running redis_worker. Also, I've made the redis_queue message configurable with confirmation_message. Hope that's fine. Even though it's throwing error. The feedbacks would be saved here => http://localhost:8000/admin/feedback/feedback/
|
PLEASE START REVIEWING. I STILL NEED TO DO SOME MORE REFACTORING. THIS PR IS RAISED FOR SUGGESTIONS ON THE CURRENT IMPLEMENTATION
Issue Reference
This PR addresses the Issue : Fixes #
Summarize
Please, describe what the PR does, the changes you have made.
Usage