Skip to content

Conversation

@tuan-nguyen-fenwick
Copy link
Contributor

@tuan-nguyen-fenwick tuan-nguyen-fenwick commented Dec 18, 2025

Prerequisite

BusinessCentralApps PR #1625

Summary

  • Updated internal description for Financial Reports, Row Definitions & Column Definitions (W1).
    • All financial reports, row & column definitions.
  • Updated internal description for Financial Reports, Row Definitions & Column Definitions (US).
    • Financial reports: BS, BS AUDIT, IS, IS AUDIT.
    • Row definitions: BS, BS AUDIT, IS, IS AUDIT.
    • Column definitions: PTD + YTD, PTD+YTD+%, YTDCOMPARE.
  • Increased field length of Internal Description parameters from Text[250] to Text[500].

Work Item(s)

Fixes: #29533
Fixes: AB#614300

@github-actions
Copy link
Contributor

Could not find linked issues in the pull request description. Please make sure the pull request description contains a line that contains 'Fixes #' followed by the issue number being fixed. Use that pattern for every issue you want to link.

@tuan-nguyen-fenwick tuan-nguyen-fenwick marked this pull request as ready for review December 18, 2025 09:50
@tuan-nguyen-fenwick tuan-nguyen-fenwick requested a review from a team as a code owner December 18, 2025 09:50
@JesperSchulz JesperSchulz added the Finance GitHub request for Finance area label Jan 6, 2026
@JesperSchulz JesperSchulz self-assigned this Jan 6, 2026
@JesperSchulz JesperSchulz added the processing-PR The PR is currently being reviewed label Jan 6, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

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

@JesperSchulz
Copy link
Contributor

I tried to fix the small issues I noted, and now we're again getting BuildDatabase issues (so far in CH, CZ and IN, but all other localizations were aborted before they could finish):

[2026-01-06 17:35:45] Invoke-jobNAVCodeunit: The length of the string is 473, but it must be less than or equal to 250 characters. Value: Eight-row layout
--------------------- calculating key metrics for cash cycle analysis, including total revenue, receivables, payables, and inventory
--------------------- balances. This layout incorporates formulas to derive days sales outstan...  CorrelationId: 
--------------------- 0c9bf769-6245-4f05-80fe-6f5a359144a7. Status: 400 (The length of the string is 473, but it must be less than or equal
--------------------- to 250 characters. Value: Eight-row layout calculating key metrics for cash cycle analysis, including total revenue,
--------------------- receivables, payables, and inventory balances. This layout incorporates formulas to derive days sales outstan...)
--------------------- ErrorCode: Application_StringExceededLength  Content:
--------------------- {"error":{"code":"Application_StringExceededLength","message":"The length of the string is 473, but it must be less
--------------------- than or equal to 250 characters. Value: Eight-row layout calculating key metrics for cash cycle analysis, including
--------------------- total revenue, receivables, payables, and inventory balances. This layout incorporates formulas to derive days sales
--------------------- outstan...  CorrelationId:  0c9bf769-6245-4f05-80fe-6f5a359144a7."}}  Headers: Server: Microsoft-HTTPAPI/2.0
--------------------- request-id: 0c9bf769-6245-4f05-80fe-6f5a359144a7 WWW-Authenticate: Negotiate oRswGaADCgEAoxIEEAEAAACPrzlNR6f5oQAAAAA=
--------------------- Date: Tue, 06 Jan 2026 16:35:30 GMT Content-Type: application/json Content-Length: 417

@tuan-nguyen-fenwick
Copy link
Contributor Author

I tried to fix the small issues I noted, and now we're again getting BuildDatabase issues (so far in CH, CZ and IN, but all other localizations were aborted before they could finish):

[2026-01-06 17:35:45] Invoke-jobNAVCodeunit: The length of the string is 473, but it must be less than or equal to 250 characters. Value: Eight-row layout
--------------------- calculating key metrics for cash cycle analysis, including total revenue, receivables, payables, and inventory
--------------------- balances. This layout incorporates formulas to derive days sales outstan...  CorrelationId: 
--------------------- 0c9bf769-6245-4f05-80fe-6f5a359144a7. Status: 400 (The length of the string is 473, but it must be less than or equal
--------------------- to 250 characters. Value: Eight-row layout calculating key metrics for cash cycle analysis, including total revenue,
--------------------- receivables, payables, and inventory balances. This layout incorporates formulas to derive days sales outstan...)
--------------------- ErrorCode: Application_StringExceededLength  Content:
--------------------- {"error":{"code":"Application_StringExceededLength","message":"The length of the string is 473, but it must be less
--------------------- than or equal to 250 characters. Value: Eight-row layout calculating key metrics for cash cycle analysis, including
--------------------- total revenue, receivables, payables, and inventory balances. This layout incorporates formulas to derive days sales
--------------------- outstan...  CorrelationId:  0c9bf769-6245-4f05-80fe-6f5a359144a7."}}  Headers: Server: Microsoft-HTTPAPI/2.0
--------------------- request-id: 0c9bf769-6245-4f05-80fe-6f5a359144a7 WWW-Authenticate: Negotiate oRswGaADCgEAoxIEEAEAAACPrzlNR6f5oQAAAAA=
--------------------- Date: Tue, 06 Jan 2026 16:35:30 GMT Content-Type: application/json Content-Length: 417

@JesperSchulz The build failed because BusinessCentralApps PR #1625 hasn't been merged into main.

@JesperSchulz
Copy link
Contributor

1625

OK, I'll see if I can merge that one.

@tuan-nguyen-fenwick
Copy link
Contributor Author

@JesperSchulz Do you have any other comment here? Thanks

@JesperSchulz
Copy link
Contributor

@JesperSchulz Do you have any other comment here? Thanks

Trying to run again. I think this one fell of my radar briefly.

@JesperSchulz
Copy link
Contributor

Internal merge completed.

@JesperSchulz JesperSchulz merged commit e6baac0 into microsoft:main Jan 29, 2026
9 of 10 checks passed
@tuan-nguyen-fenwick tuan-nguyen-fenwick deleted the task/update-internal-descriptions-500 branch January 29, 2026 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Finance GitHub request for Finance area processing-PR The PR is currently being reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Internal descriptions for report/row/column definitions with Text[500]

2 participants