Skip to content

[Master] [Repair Item] Team Member unable to copy sales quotation when lines are associated to assembly - #9644

Open
Shikhverma wants to merge 10 commits into
mainfrom
bugs/Bug-630947-Master-TeamMemberUnableToCopySaleQuoteWhenAssemblyItemUsed
Open

[Master] [Repair Item] Team Member unable to copy sales quotation when lines are associated to assembly#9644
Shikhverma wants to merge 10 commits into
mainfrom
bugs/Bug-630947-Master-TeamMemberUnableToCopySaleQuoteWhenAssemblyItemUsed

Conversation

@Shikhverma

@Shikhverma Shikhverma commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Bug 630947: [ALL-E] [Repair Item] Team Member unable to copy sales quotation when lines are associated to assembly

Fixes AB#630947

@github-actions github-actions Bot added the Integration GitHub request for Integration area label Jul 21, 2026
@github-actions github-actions Bot added this to the Version 29.0 milestone Jul 21, 2026
@Shikhverma Shikhverma added the SCM GitHub request for SCM area label Jul 22, 2026
@github-actions github-actions Bot removed the SCM GitHub request for SCM area label Jul 22, 2026
@Shikhverma Shikhverma added the SCM GitHub request for SCM area label Jul 22, 2026
@github-actions github-actions Bot removed the SCM GitHub request for SCM area label Jul 22, 2026
Comment thread src/Layers/W1/BaseApp/Utilities/CopyDocumentMgt.Codeunit.al Outdated
…Bug-630947-Master-TeamMemberUnableToCopySaleQuoteWhenAssemblyItemUsed
@Shikhverma Shikhverma closed this Jul 28, 2026
@Shikhverma Shikhverma reopened this Jul 28, 2026
@Shikhverma Shikhverma closed this Jul 28, 2026
@Shikhverma Shikhverma reopened this Jul 28, 2026
@Shikhverma Shikhverma added the SCM GitHub request for SCM area label Jul 28, 2026
@github-actions github-actions Bot removed the SCM GitHub request for SCM area label Jul 28, 2026
@Shikhverma Shikhverma closed this Jul 28, 2026
@Shikhverma Shikhverma reopened this Jul 28, 2026
Comment thread src/Layers/RU/BaseApp/Utilities/CopyDocumentMgt.Codeunit.al
Comment thread src/Layers/RU/BaseApp/Utilities/CopyDocumentMgt.Codeunit.al
Comment thread src/Layers/W1/BaseApp/Utilities/CopyDocumentMgt.Codeunit.al
Comment thread src/Layers/W1/BaseApp/Utilities/CopyDocumentMgt.Codeunit.al

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

S1 (High): This permission fix has no regression test under the restricted Team Member entitlement. Full-permission test runs cannot demonstrate that copying a sales quotation with an assemble-to-order line now succeeds, or that the RIM grants are sufficient. Please add an automated scenario that executes the copy with the Team Member permission set and verifies the quotation and associated assembly data are copied successfully.

Comment thread src/Layers/W1/Tests/SCM/SCMCopyDocumentMgt.Codeunit.al
@Shikhverma

Copy link
Copy Markdown
Contributor Author

S1 (High): This permission fix has no regression test under the restricted Team Member entitlement. Full-permission test runs cannot demonstrate that copying a sales quotation with an assemble-to-order line now succeeds, or that the RIM grants are sufficient. Please add an automated scenario that executes the copy with the Team Member permission set and verifies the quotation and associated assembly data are copied successfully.

done Changes

Comment thread src/Layers/W1/BaseApp/Utilities/CopyDocumentMgt.Codeunit.al
v-ankitgoyal
v-ankitgoyal previously approved these changes Aug 12, 2026
Comment thread src/Layers/W1/Tests/SCM/SCMCopyDocumentMgt.Codeunit.al Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved with one follow-up suggestion (S1): strengthen the regression coverage so it exercises the Team Member plan/entitlement, which is the production boundary this fix intentionally overrides.

Comment thread src/Layers/W1/BaseApp/Utilities/CopyDocumentMgt.Codeunit.al
@Shikhverma Shikhverma closed this Aug 13, 2026
auto-merge was automatically disabled August 13, 2026 16:45

Pull request was closed

@Shikhverma Shikhverma reopened this Aug 13, 2026
Comment thread src/Layers/W1/BaseApp/Utilities/CopyDocumentMgt.Codeunit.al
@Shikhverma
Shikhverma enabled auto-merge August 14, 2026 04:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Integration GitHub request for Integration area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants