Skip to content

[master] [NAV] Track failed BCApps validation (AUTODETECTED) - Shopify test updated#9406

Open
djukicmilica wants to merge 5 commits into
mainfrom
bugs/ShopifyUpdate
Open

[master] [NAV] Track failed BCApps validation (AUTODETECTED) - Shopify test updated#9406
djukicmilica wants to merge 5 commits into
mainfrom
bugs/ShopifyUpdate

Conversation

@djukicmilica

@djukicmilica djukicmilica commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

What & why

Linked work

Fixes #
AB#642193

How I validated this

  • I read the full diff and it contains only changes I intended.
  • I built the affected app(s) locally with no new analyzer warnings.
  • I ran the change in Business Central and confirmed it behaves as expected.
  • I added or updated tests for the new behavior, or explained below why none are needed.

What I tested and the outcome (required — be specific: scenarios, commands, screenshots for UI changes)

Risk & compatibility

@djukicmilica djukicmilica requested review from a team July 14, 2026 10:10
@djukicmilica djukicmilica enabled auto-merge July 14, 2026 10:14
@github-actions github-actions Bot added this to the Version 29.0 milestone Jul 14, 2026
@github-actions

github-actions Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Copilot PR Review

Iteration 3 · 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.

auto-merge was automatically disabled July 15, 2026 08:53

Pull request was closed

@djukicmilica djukicmilica reopened this Jul 15, 2026
@djukicmilica djukicmilica enabled auto-merge July 15, 2026 08:53
auto-merge was automatically disabled July 16, 2026 07:35

Pull request was closed

@djukicmilica djukicmilica reopened this Jul 16, 2026
@djukicmilica djukicmilica enabled auto-merge July 16, 2026 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants