Skip to content

Withholding Tax for Employee#9311

Draft
ViditGupta1277 wants to merge 10 commits into
microsoft:mainfrom
ViditGupta1277:features/Slice-580268-Idea]-VENDOR]-WHT-for-Employee
Draft

Withholding Tax for Employee#9311
ViditGupta1277 wants to merge 10 commits into
microsoft:mainfrom
ViditGupta1277:features/Slice-580268-Idea]-VENDOR]-WHT-for-Employee

Conversation

@ViditGupta1277

@ViditGupta1277 ViditGupta1277 commented Jul 10, 2026 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Contributor

Slices AB#580268

@github-actions github-actions Bot added From Fork Pull request is coming from a fork Linked Issue is linked to a Azure Boards work item labels Jul 10, 2026
@github-actions github-actions Bot added this to the Version 29.0 milestone Jul 10, 2026
@github-actions github-actions Bot added the needs-approval Workflow runs require maintainer approval to start label Jul 10, 2026
@github-actions github-actions Bot added the AL: Apps (W1) Add-on apps for W1 label Jul 13, 2026
@aholstrup1

aholstrup1 commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Heads up, these build failures aren't from your code changes. The special character (a stray ]) in the branch name trips up our pipeline. Recreating the branch without the bracket and repointing the PR should fix it. Sticking to letters, numbers, /, -, and _ in branch names avoids this going forward.

BardurKnudsen
BardurKnudsen reviewed Jul 14, 2026
View reviewed changes
Comment thread src/Apps/W1/WithholdingTax/app/src/Employee/History/WHTEmplLedgerEntryExt.TableExt.al Outdated
Comment thread src/Apps/W1/WithholdingTax/app/src/Employee/Master/WHTEmployeeCard.PageExt.al Outdated
Comment thread src/Apps/W1/WithholdingTax/app/src/Employee/Master/WHTEmployeeCard.PageExt.al
Comment thread src/Apps/W1/WithholdingTax/app/src/Employee/Transaction/WthldgTaxEmplSubscribers.Codeunit.al Outdated
Comment thread src/Apps/W1/WithholdingTax/app/src/Employee/Transaction/WthldgTaxEmplSubscribers.Codeunit.al
Comment thread src/Apps/W1/WithholdingTax/app/src/Employee/Setup/WithholdingTaxGroupLine.Table.al
@JesperSchulz JesperSchulz added Finance GitHub request for Finance area Ownership: Manual Preserve the manually selected team ownership labels Jul 17, 2026
Alexander-Ya
Alexander-Ya reviewed Jul 20, 2026
View reviewed changes
Comment thread src/Apps/W1/WithholdingTax/app/src/Employee/Setup/WithholdingTaxGroupLine.Table.al Outdated
Alexander-Ya
Alexander-Ya reviewed Jul 23, 2026
View reviewed changes
Comment thread src/Apps/W1/ExpenseWithholdingTax/app/app.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@BardurKnudsen BardurKnudsen BardurKnudsen left review comments
@Alexander-Ya Alexander-Ya Alexander-Ya approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

AL: Apps (W1) Add-on apps for W1 Finance GitHub request for Finance area From Fork Pull request is coming from a fork Linked Issue is linked to a Azure Boards work item needs-approval Workflow runs require maintainer approval to start Ownership: Manual Preserve the manually selected team ownership

Projects

None yet

Milestone

Version 29.0

Development

Successfully merging this pull request may close these issues.

5 participants

@ViditGupta1277 @aholstrup1 @BardurKnudsen @Alexander-Ya @JesperSchulz