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

[16.0][MIG] pos_ticket_extra_company_info #1131

Draft
wants to merge 8 commits into
base: 16.0
Choose a base branch
from

Conversation

legalsylvain
Copy link
Contributor

@legalsylvain legalsylvain commented Jan 17, 2024

Migration of the module pos_ticket_extra_company_info from V12.
Note There are some simplification, because less fields are missing.

Question : I was unable to display an html field on the receipt (for the address). If anyone can help, I'm interested. (not a blocking point though).

Description

This module extends the functionality of Point of Sale module.

It adds missing company fields on PoS ticket. For instance:

  • company address
  • company registry

@legalsylvain
Copy link
Contributor Author

/ocabot migration pos_ticket_extra_company_info

@OCA OCA deleted a comment from OCA-git-bot Jan 17, 2024
@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Jan 17, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Jan 17, 2024
37 tasks
@legalsylvain legalsylvain force-pushed the 16.0-mig-pos_ticket_extra_company_info branch from 717e789 to 1d26978 Compare January 18, 2024 08:29
@legalsylvain
Copy link
Contributor Author

Hi @OCA/pos-maintainers : I have a question on that PR.
I try to display an html field. (company address for instance) on the ticket, but the html tags are not interpretated.
See :

image

XML code : <t t-raw="receipt.company.pos_ticket_address" />

python code : pos_ticket_address = fields.Html(compute="_compute_pos_ticket_address")

Ref Odoo Doc : https://www.odoo.com/documentation/16.0/developer/tutorials/web.html?highlight=modify%20qweb#outputting-html

@legalsylvain legalsylvain marked this pull request as draft January 18, 2024 08:31
@chienandalu
Copy link
Member

XML code :
python code : pos_ticket_address = fields.Html(compute="_compute_pos_ticket_address")

Have you tried with t-out?

@legalsylvain
Copy link
Contributor Author

Have you tried with t-out?

Thanks for your time, @chienandalu.
Yes, without success.

See :

image

Copy link

@quentinDupont quentinDupont left a comment

Choose a reason for hiding this comment

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

Functional review ✔️
Except html issue it works well :)

Copy link

github-actions bot commented Jun 2, 2024

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jun 2, 2024
@OCA OCA deleted a comment from OCA-git-bot Jul 2, 2024
@legalsylvain legalsylvain force-pushed the 16.0-mig-pos_ticket_extra_company_info branch from 1d26978 to 4e62a6c Compare July 2, 2024 09:29
@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jul 21, 2024
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.

5 participants