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

[Bug]: Duplicated query on codeunit "Copy Document Mgt.", function CopySalesShptLinesToDoc #26954

Open
1 task done
GiangLNT56 opened this issue Aug 7, 2024 · 1 comment
Open
1 task done
Labels
bug Something isn't working follow up Follow up is needed SCM GitHub request for SCM area

Comments

@GiangLNT56
Copy link

GiangLNT56 commented Aug 7, 2024

Describe the issue

Hi team,

In Codeunit "Copy Document", function CopySalesShptLinesToDoc, I see 2 lines testing "Price Including VAT" as below.

2024-08-05_17h07_36

Expected behavior

I think we should remove the second line.

Steps to reproduce

N/A

Additional context

No response

I will provide a fix for a bug

  • I will provide a fix for a bug
@GiangLNT56 GiangLNT56 added the bug Something isn't working label Aug 7, 2024
@JesperSchulz
Copy link
Contributor

Sounds fair, but since the code you are referring to is in the BaseApp, you'd have to suggest and fix this code in the BaseApp contribution pilot: https://github.com/microsoft/businesscentralapps. For access to the pilot, please fill out the following form: https://forms.office.com/r/E9wyE5a58f.

This repository is only for extensibility requests or pull requests towards the 1st party Business Central apps.

Would you like me to move the issue to the contribution pilot for you?

IMPORTANT: Microsoft is not going to address the reported issue for you. You will have to get the issue approved, and then push a pull request with the suggested changes yourself. If you would like Microsoft to address the issue, please create a support request. Learn more about our support channels here: https://aka.ms/bcsupport.

@JesperSchulz JesperSchulz added the follow up Follow up is needed label Aug 26, 2024
@JesperSchulz JesperSchulz added the SCM GitHub request for SCM area label Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working follow up Follow up is needed SCM GitHub request for SCM area
Projects
None yet
Development

No branches or pull requests

2 participants