Skip to content

Number of Lines in Batches#29697

Merged
JesperSchulz merged 2 commits intomicrosoft:mainfrom
pri-kise:115-number-of-lines-in-batch
Feb 26, 2026
Merged

Number of Lines in Batches#29697
JesperSchulz merged 2 commits intomicrosoft:mainfrom
pri-kise:115-number-of-lines-in-batch

Conversation

@pri-kise
Copy link
Contributor

@pri-kise pri-kise commented Feb 12, 2026 •
edited by JesperSchulz
Loading

Summary

Add Field "No. of Lines" to the Journal Batch Tables.

Work Item(s)

Fixes #29696
Fixes https://github.com/microsoft/BusinessCentralApps/issues/115
Fixes AB#623062

@pri-kise pri-kise requested a review from a team as a code owner February 12, 2026 07:13
@pri-kise pri-kise requested a review from haoranpb February 12, 2026 07:13
@github-actions
Copy link
Contributor

github-actions bot commented Feb 12, 2026

Issue #29696 is not valid. Please make sure you link an issue that exists, is open and is approved.

@JesperSchulz JesperSchulz added the Finance GitHub request for Finance area label Feb 25, 2026
@JesperSchulz JesperSchulz self-assigned this Feb 25, 2026
@JesperSchulz JesperSchulz added linked Issue is linked to a Azure Boards work item processing-PR The PR is currently being reviewed labels Feb 25, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 25, 2026

Processing this PR. The branch is now locked 🔒 Please don't push updates unless otherwise agreed.

@JesperSchulz
Copy link
Contributor

JesperSchulz commented Feb 26, 2026

Build is also failing internally with PTE0008 Field 'No. of Lines' must have a value for the ApplicationArea property so that they are visible to users.

I've added ApplicationArea where it was missing and will try again. If it passes, we'll just close this one and sync out the new changes.

JesperSchulz
JesperSchulz previously approved these changes Feb 26, 2026
View reviewed changes
JesperSchulz
JesperSchulz reviewed Feb 26, 2026
View reviewed changes
Copy link
Contributor

@JesperSchulz JesperSchulz left a comment

Choose a reason for hiding this comment

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

Here the fixes I made internally.

@JesperSchulz JesperSchulz enabled auto-merge (squash) February 26, 2026 17:05
@JesperSchulz JesperSchulz merged commit a338604 into microsoft:main Feb 26, 2026
10 of 12 checks passed
@JesperSchulz JesperSchulz added the ships-in-future-update Fix ships in a future update label Feb 27, 2026
@JesperSchulz
Copy link
Contributor

JesperSchulz commented Feb 27, 2026

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Build ID: 46795.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JesperSchulz JesperSchulz JesperSchulz approved these changes

@djukicmilica djukicmilica djukicmilica approved these changes

@haoranpb haoranpb Awaiting requested review from haoranpb haoranpb is a code owner automatically assigned from microsoft/dynamics-365-business-central

Assignees

@JesperSchulz JesperSchulz

Labels

Finance GitHub request for Finance area linked Issue is linked to a Azure Boards work item processing-PR The PR is currently being reviewed ships-in-future-update Fix ships in a future update

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[BC Idea]: Number of Lines in Batches

3 participants

@pri-kise @JesperSchulz @djukicmilica