Skip to content
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

customer listing in ticket activity #1854

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

NihalRoshanCK
Copy link

customer listing fixed at the ticket activity right sidebar
helpdesk_bug

@NihalRoshanCK
Copy link
Author

when we create a new ticket from helpdesk we can't add customer reference after creating that we have a view that shows all the ticket details in its sidebar there are details including customer and all if there is no customer (in case we create it from helpdesk )
from there customer listing is not working

@rmehta
Copy link
Member

rmehta commented Jul 15, 2024

can you send a clear video?

@NihalRoshanCK
Copy link
Author

NihalRoshanCK commented Jul 15, 2024

helpdeskuserlist.mp4

if we want to add a reference of user in ticket user is not listing the sidebar

@sumitbhanushali
Copy link
Collaborator

@Nihalroshan55 in your change, you are fetching data from Customer in agentstore? was this by mistake?

Also, Customers are fetched based on Contact -> Customer mapping. If a contact doesn't have customer mapped, It shows blank. This thing needs better UX but no change in functionality afaik

@NihalRoshanCK
Copy link
Author

yes, it's done by mistake changed it back @sumitbhanushali

@sumitbhanushali
Copy link
Collaborator

@Nihalroshan55 checkout get_customer api at helpdesk/utils.py.

basically, it fetches customer from given Contact. This contact must have mapping in Contact doctype in links child table

@NihalRoshanCK
Copy link
Author

as in our use case, we received a call for a product bug, which we are trying to manually create a ticket on behalf of customer. Here we would like to create a customer manually or sync from our customer list. (this is a non frappe external use case)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants