Skip to content

[ADD] point_of_sale: added salesperson field #897

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

Draft
wants to merge 1 commit into
base: 18.0
Choose a base branch
from

Conversation

kdes-odoo
Copy link

@kdes-odoo kdes-odoo commented Jul 28, 2025

Introduced the functionality to assign a salesperson to a Point of Sale order.

Added new dialog menu, populated with employees from hr.employee, has been added
to the main POS interface.

A (Many2one) field was added to the model to store
the selected employee.

The POS Order form and list views in the backend have been updated to display the salesperson's name.

Employee data is now loaded into the POS session to make it available in the UI.

Task-4968717

@robodoo
Copy link

robodoo commented Jul 28, 2025

Pull request status dashboard

Copy link

@deso-odoo deso-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some remarks!
Also please add task id.

Introduced the functionality to assign a salesperson to a Point of Sale order.

Added new dialog menu, populated with employees from hr.employee, has been added
 to the main POS interface.

A  (Many2one) field was added to the  model to store
 the selected employee.

The POS Order form and list views in the backend have been updated to display
the salesperson's name.

Employee data is now loaded into the POS session to make it available in the UI.

task-4968717
@kdes-odoo kdes-odoo force-pushed the 18.0-training-task-pos_salesperson-kdes branch from 738b74a to a03981a Compare August 4, 2025 05:42
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