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

FIX TakePOS receipt preview in admin #25648

Merged
merged 1 commit into from
Aug 19, 2023
Merged

FIX TakePOS receipt preview in admin #25648

merged 1 commit into from
Aug 19, 2023

Conversation

Max13
Copy link
Contributor

@Max13 Max13 commented Aug 19, 2023

Receipt preview in admin shows header and footer repeating themselves because in preview, $constFreeText equals TAKEPOS_HEADER (as there is no $_SESSION['takeposterminal'] defined).

Capture d’écran 2023-08-19 à 12 18 33 Capture d’écran 2023-08-19 à 12 18 41

Receipt preview in admin shows header and footer repeating themselves because in preview, `$constFreeText` equals `TAKEPOS_HEADER` (as there is no `$_SESSION['takeposterminal']` defined).
@eldy eldy merged commit b426478 into Dolibarr:18.0 Aug 19, 2023
2 checks passed
@Max13
Copy link
Contributor Author

Max13 commented Aug 19, 2023

@eldy Should I also PR the same fix for 16.0, 17.0 and develop ?

Max13 added a commit to Max13/dolibarr that referenced this pull request Aug 19, 2023
Receipt preview in admin shows header and footer repeating themselves because in preview, $constFreeText equals TAKEPOS_HEADER (as there is no $_SESSION['takeposterminal'] defined).

Back compatibility for Dolibarr#25648
Max13 added a commit to Max13/dolibarr that referenced this pull request Aug 19, 2023
Receipt preview in admin shows header and footer repeating themselves because in preview, $constFreeText equals TAKEPOS_HEADER (as there is no $_SESSION['takeposterminal'] defined).

Back compatibility for Dolibarr#25648
Max13 added a commit to Max13/dolibarr that referenced this pull request Aug 19, 2023
Receipt preview in admin shows header and footer repeating themselves because in preview, $constFreeText equals TAKEPOS_HEADER (as there is no $_SESSION['takeposterminal'] defined).

Following Dolibarr#25648
@eldy
Copy link
Member

eldy commented Aug 19, 2023

When a fix is pushed for version x, it is also available for version x+1, x+2, ... and develop. Merges are done regularly to have fixes propagated.
So you can push it on v16 bit no need to do it for v17 .

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.

None yet

2 participants