Skip to content

Dynamics BC Excel Reports Docs#7269

Open
Groenbech96 wants to merge 3 commits intomainfrom
docs/al-docs/excel-reports
Open

Dynamics BC Excel Reports Docs#7269
Groenbech96 wants to merge 3 commits intomainfrom
docs/al-docs/excel-reports

Conversation

@Groenbech96
Copy link
Copy Markdown
Contributor

@Groenbech96 Groenbech96 commented Mar 19, 2026

Summary

  • Bootstrap documentation for Dynamics BC Excel Reports
  • Generated by /al-docs init

🤖 Generated with Claude Code

AB#626091

@Groenbech96 Groenbech96 requested a review from a team as a code owner March 19, 2026 13:49
@github-actions github-actions bot added the AL: Apps (W1) Add-on apps for W1 label Mar 19, 2026

- **`TrialBalance` codeunit (4410)** -- The shared engine behind 4 of
the 5 trial balance reports. Has a dual-path architecture: a
query-based path (default from v27+) and a legacy looping path
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.

(default from v27+)

not true

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@copilot Can you take a look

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@copilot Can you take a look

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.

Fixed in c23faa2. The documentation in both CLAUDE.md and docs/business-logic.md incorrectly described the query-based path as feature-flag-gated in v28+. In reality, once CLEAN27 is defined (v28+), the query path is always active with no feature flag involved — the feature flag only switched between paths in pre-v28 builds. Also corrected the "When CLEAN27 ships" language since it's already defined.

Add AL documentation for the Excel Reports app:
- CLAUDE.md: App overview, report types, temporary buffer architecture
- docs/data-model.md: 4 temp buffer tables, 7 queries, aging/TB/ranking design
- docs/business-logic.md: Trial balance engine, aging bucketing, top-N ranking
- docs/extensibility.md: Caption events, OnOverrideAgedBy, manual subscription
- docs/patterns.md: Temp buffers, query aggregation, feature flags, debit/credit split
@Groenbech96 Groenbech96 force-pushed the docs/al-docs/excel-reports branch from 4e82110 to b3e720b Compare March 19, 2026 20:44
@Groenbech96 Groenbech96 changed the title Add AL docs for Dynamics BC Excel Reports Dynamics BC Excel Reports Docs Mar 19, 2026
@github-actions github-actions bot added this to the Version 29.0 milestone Mar 20, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

⚠️ Stale Status Check Deleted

The Pull Request Build workflow run for this PR was older than 72 hours and has been deleted.

📋 Why was it deleted?

Status checks that are too old may no longer reflect the current state of the target branch. To ensure this PR is validated against the latest code and passes up-to-date checks, a fresh build is required.


🔄 How to trigger a new status check:

  1. 📤 Push a new commit to the PR branch, or
  2. 🔁 Close and reopen the PR

This will automatically trigger a new Pull Request Build workflow run.

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

Labels

AL: Apps (W1) Add-on apps for W1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants