Fixes to the Advisors' page

PaulaMerle commented 1 year ago

Tested at https://admin.dev.buerokratt.ee/analytics/advisors

[ ] Forwarding - currently the result number for each selection (chats forwarded to CSA, chats forwarded by CSA or chats forwarded to other organization) is identical on the graph. As of now, result for Chats Forwarded to other organization should always be 0, since it is not possible yet to forward chats to another organization. Also the forwarded to and forwarded by cannot always be identical numbers.
[x] Average response speed in the organization - the result of the metric should be displayed in minutes with 2 decimals and should actually show the same result as the average wait time on the chats page. Name of y-axe should be corrected accordingly.
[x] Conversations by advisor - the graph should show the number of conversations for each advisor separately with the advisor's name. Currently only one number is displayed.
[x] Chat time by advisor - similarly should display the average chat time by each advisor and check that the metric shows minutes. Name of y-axe should be corrected accordingly.

PaulaMerle commented 1 year ago

@1AhmedYasser explanation for metric Forwarding. It would be more clear for the user if each option (lets name them here as to, from and outside) is displayed as separate column. The To column shows how many conversations have been directed to each advisor (name and number of chats. like in Feedback to selected CSA) during the selected period. The From column shows the same way, how many conversations each CSA has forwarded to another CSA, and the Outside column shows as a single number, how many conversations have been directed from the agency to other agencies.

PaulaMerle commented 1 year ago

BUG - tested at https://admin.dev.buerokratt.ee/analytics/advisors Problem - additional options under metric Forwarding cannot be unselected Expected result - the user must be able to select and unselect additional options

PaulaMerle commented 1 year ago

BUG - conversations by advisor Currently: column legend gives only one name for each day, also name and number of chats is separated Expected result: legend should show the information as follows: name of CSA: number of chats (Rasmuss: 34, Jana: 15). like in Feedback to selected CSA.

ValterAro commented 11 months ago

BUG - conversations by advisor Currently: column legend gives only one name for each day, also name and number of chats is separated Expected result: legend should show the information as follows: name of CSA: number of chats (Rasmuss: 34, Jana: 15). like in Feedback to selected CSA.

The following bug is currently not correctly fixed. Short summary: Graphs for "conversations by advisors" and "conversation time by advisors" do show several csas for a data point(08-11-2023 for "conversation time by advisors" and "conversation time by advisors"), but not in a correct way. If you are on the left side of the column then data for Janina is shown and when on the right side of the column then data for OK is shown.

Pic of one of the graphs:

1AhmedYasser commented 11 months ago

BUG - conversations by advisor Currently: column legend gives only one name for each day, also name and number of chats is separated Expected result: legend should show the information as follows: name of CSA: number of chats (Rasmuss: 34, Jana: 15). like in Feedback to selected CSA.

The following bug is currently not correctly fixed. Short summary: Graphs for "conversations by advisors" and "conversation time by advisors" do show several csas for a data point(08-11-2023 for "conversation time by advisors" and "conversation time by advisors"), but not in a correct way. If you are on the left side of the column then data for Janina is shown and when on the right side of the column then data for OK is shown.

Pic of one of the graphs:

@ValterAro as a prior discussion with @PaulaMerle regarding it, the main version was only calculating conversation time by advisors in total, and i enhanced it to this to show the CSA name when you hover on the bars, i am aware further enhancement is needed here but its currently out of scope and needs to be separated in a separate task @PaulaMerle what is your insights on this, how do you suggest we move forward with it

PaulaMerle commented 10 months ago

For the last comment new task is created https://github.com/buerokratt/Analytics-Module/issues/144

ValterAro commented 9 months ago

The business logic for the "Suunamised" will be revised by @PaulaMerle

PaulaMerle commented 9 months ago

Done with https://github.com/buerokratt/Analytics-Module/issues/144

buerokratt / Analytics-Module

Fixes to the Advisors' page #112