-
Notifications
You must be signed in to change notification settings - Fork 518
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
[Reporting - API] Add method to count messages sent/received by date #255
Comments
As mentioned in #254 I'd like to pick this up 👍 |
I'd like to help with this one, is it available? I saw that there was nos activity over the past 4 days. |
Wow ok, thanks! |
Totally fine. It is taking me some time to set up the Elixir development environment for some reason, so I am still working on #254 :-) |
@eikooc If you need some help, you can call me and we can pair or something to set the environment and discuss about the task... 😸 |
btw we also have a shared slack channel if you guys want to chat there :) https://github.com/papercups-io/papercups#get-in-touch |
@WLSF that would be fun! join the slack channel and let's coordinate! 🐈 |
done in #272 |
Problem
Users should be able to see a breakdown of messages sent vs received in the reporting dashboard.
Solution
Let's create two methods in the Reporting context (https://github.com/papercups-io/papercups/blob/master/lib/chat_api/reporting.ex)
user_id
is notnull
(i.e. it was sent by a signed-in user)customer_id
is notnull
(i.e. it was sent by a customer through the chat widget)The responses should look something like this:
Testing
Please add tests to
test/chat_api/reporting_test.exs
and make sure they pass withmix test
:)Context
See #252
Questions, or need help getting started?
Feel free to ask below, or ping us on Slack :)
(You can also check out our CONTRIBUTING.md)
The text was updated successfully, but these errors were encountered: