Skip to content

[Extensibility Request] issue 30349: add OnBeforeCheckTrackingIfRequired event to Item Journal Line#9464

Open
AleksandricMarko wants to merge 1 commit into
mainfrom
ai-ext-fix/ext_issue-30349
Open

[Extensibility Request] issue 30349: add OnBeforeCheckTrackingIfRequired event to Item Journal Line#9464
AleksandricMarko wants to merge 1 commit into
mainfrom
ai-ext-fix/ext_issue-30349

Conversation

@AleksandricMarko

@AleksandricMarko AleksandricMarko commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Summary

The reporter needs clearer, user-facing error messages when item tracking (Serial/Lot No.) is missing during document posting. The standard CheckTrackingIfRequired validation reports errors against the temporary "Item Journal Line" record, so users cannot tell which document line or item is affected. This PR adds an integration event at the start of CheckTrackingIfRequired so extensions can run their own validation and raise a more meaningful error before the standard checks.

Source issue repository: microsoft/AlAppExtensions; issue number: 30349

Changes Made

  • CheckTrackingIfRequired (table 83 "Item Journal Line") - raise a new OnBeforeCheckTrackingIfRequired integration event at the start of the procedure so subscribers can perform custom item-tracking validation before the standard TestField checks; the same change is propagated to the country layer counterparts (APAC, CH, ES, FR, GB, IT, RU).

Fixes AB#642241

…m Journal Line

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@AleksandricMarko AleksandricMarko added event-request SCM GitHub request for SCM area labels Jul 15, 2026
@github-actions github-actions Bot added this to the Version 29.0 milestone Jul 15, 2026
@AleksandricMarko
AleksandricMarko marked this pull request as ready for review July 16, 2026 07:57
@AleksandricMarko
AleksandricMarko requested a review from a team July 16, 2026 07:57
@github-actions

Copy link
Copy Markdown
Contributor

Copilot PR Review

Iteration 1 · Outcome: completed

Knowledge source: https://github.com/microsoft/BCQuality@186d8a131465475c79244d994acb872cd5c0d4bf

Orchestrator pre-filter (2 file(s) excluded)

  • layer-disabled (knowledge) : 2 file(s)

Findings produced by the AL review agent v1.7.3. Reply 👎 on any inline comment to flag false positives.

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ 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.

@qasimikram qasimikram left a comment

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.

The new publisher is raised before both standard tracking validations and exposes the requested item journal line and tracking setup context. The implementation is consistent across all localized table copies and does not alter default behavior.

auto-merge was automatically disabled July 23, 2026 16:49

Pull request was closed

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

Labels

event-request SCM GitHub request for SCM area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants