Extract NL customization to object extensions (TrMode) by Alexander-Ya · Pull Request #8933 · microsoft/BCApps

Alexander-Ya · 2026-06-30T13:39:15Z

What & why

Linked work

Fixes AB#640601

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

Alexander-Ya · 2026-06-30T13:39:57Z

Fixes AB#640601

github-actions · 2026-06-30T13:54:54Z

Copilot PR Review

Iteration 17 · Outcome: completed

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

Findings by domain

Findings split into Knowledge-backed (cite a BCQuality article) and Agent (the agent's own judgement, no matching BCQuality rule).

Domain	Findings	Knowledge-backed	Agent	Inline	Fallback
Accessibility	1	1	0	0	0
Breaking Changes	2	2	0	2	0

Totals: 3 knowledge-backed · 0 agent findings.

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.}

The model-reported anchor for a single-line suggestion can be off by more than 8 lines (e.g. PR #8933 VendorNL Confirm() was off by 10, label rename off by 9). Widen the search window to 40 lines either side so the correct target within the same procedure is considered, while the 0.5 similarity floor and 0.1 ambiguity margin keep an unrelated look-alike from winning. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Widening the search window exposed a precision problem: a label-rename suggestion whose added Comment text echoes the field captions at the Error()/Confirm() call site would re-anchor onto the call site (PR #8933 GenJournalLineNL/GeneralLedgerSetupNL 'Text1000000/1' findings), which is worse than the original mis-anchor. Raise the similarity floor to 0.6 and use a bounded window (20). Genuine edit targets - an edited statement or a renamed declaration - score ~0.75-0.99 and re-anchor confidently (the PR #8933 Confirm() findings and the PartnerTypeMismatchMsg->Qst rename now land on the right line). Lower- confidence look-alikes stay below the floor and are suppressed, so the caller posts a manual snippet instead of a wrong auto-applicable anchor. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…Extract-NL-customization-to-object-extensions-v2

github-actions · 2026-07-17T09:45:39Z

+    Access = Internal;
+
+    [EventSubscriber(ObjectType::Codeunit, Codeunit::"Sales-Post Prepayments", 'OnBeforePostCustomerEntry', '', false, false)]
+    local procedure OnBeforePostCustomerEntry(var GenJnlLine: Record "Gen. Journal Line"; SalesHeader: Record "Sales Header")


$\textbf{🔴\ Critical\ Severity\ —\ Style} \quad \color{gray}{\texttt{\small Iteration\ 16}}$

The new EventSubscriber OnBeforePostCustomerEntry on Codeunit::"Sales-Post Prepayments" is declared with only two parameters, `(var GenJnlLine: Record "Gen.

Journal Line"; SalesHeader: Record "Sales Header"), but the publisher (SalesPostPrepayments.Codeunit.al line 2431) declares six parameters: (var GenJnlLine: Record "Gen. Journal Line"; TotalPrepmtInvLineBuffer: Record "Prepayment Inv. Line Buffer"; TotalPrepmtInvLineBufferLCY: Record "Prepayment Inv. Line Buffer"; CommitIsSuppressed: Boolean; SalesHeader: Record "Sales Header"; DocumentType: Option Invoice,"Credit Memo")`. A subscriber cannot cherry-pick a subset of the publisher's parameters; the full parameter list, in order, must be reproduced. As written this subscriber will not bind to the publisher and the extracted NL Transaction Mode Code/Bank Account Code logic that used to run inline in the old NL override of Sales-Post Prepayments will silently stop firing.

Knowledge:

microsoft/knowledge/style/event-subscriber-param-names-match-publisher.md

_{👍 useful · ❤️ especially valuable · 👎 wrong - reply with why}

Sign in to view

@@ -7581,7 +7582,7 @@ table 36 "Sales Header"
        if ReplaceVATDate then
            "VAT Reporting Date" := VATDateReq;

-        if ReplaceDocDate and ("Document Date" <> PostingDateReq) then begin
+        if ReplacePostingDate and ReplaceDocDate and ("Document Date" <> PostingDateReq) then begin


…Extract-NL-customization-to-object-extensions-v2

github-actions · 2026-07-18T10:46:10Z

+        }
+    }
+
+    procedure UpdateElecTaxDeclCategoryCode()


$\textbf{🟡\ Medium\ Severity\ —\ Breaking\ Changes} \quad \color{gray}{\texttt{\small Iteration\ 17}}$

VATStatementLineNL.TableExt widens UpdateElecTaxDeclCategoryCode from a local procedure in the original full-object NL override to a public tableextension procedure, but it is only invoked from the companion VATStatementLineNL.Codeunit.al within the same app.

Publishing it needlessly turns an internal helper into an external contract that must now be preserved across future refactors.

Suggested change

procedure UpdateElecTaxDeclCategoryCode()

internal procedure UpdateElecTaxDeclCategoryCode()

Knowledge:

microsoft/knowledge/breaking-changes/choose-access-modifiers-deliberately.md

_{👍 useful · ❤️ especially valuable · 👎 wrong - reply with why}

github-actions · 2026-07-18T10:46:12Z

+        UpdateBankAccountsQst: Label 'Do you want to update the bank accounts for this customer to reflect the new value of %1?', Comment = '%1 = Field Caption';
+        PartnerTypeMismatchQst: Label 'The Partner Type does not match the Partner Type defined in Transaction Mode. Do you still want to change the Partner Type?';
+
+    [Scope('OnPrem')]


$\textbf{🟡\ Medium\ Severity\ —\ Breaking\ Changes} \quad \color{gray}{\texttt{\small Iteration\ 17}}$

CustomerNL.TableExt widens UpdateCustomerBankAccounts from a local procedure in the original full-object NL override to a published, non-local procedure marked only with [Scope('OnPrem')], even though every call site remains inside the same tableextension.

That turns an implementation detail into a supported API surface for no functional reason, freezing a contract that was never meant to be external.

Suggested fix (apply manually — could not be anchored as a one-click suggestion):

[Scope('OnPrem')] local procedure UpdateCustomerBankAccounts(UseFieldCaption: Text[250])

Knowledge:

microsoft/knowledge/breaking-changes/choose-access-modifiers-deliberately.md

_{👍 useful · ❤️ especially valuable · 👎 wrong - reply with why}

…Extract-NL-customization-to-object-extensions-v2

github-actions · 2026-07-20T11:47:55Z

+    {
+        addafter("Industrial Classification")
+        {
+            field("Fiscal Entity No."; Rec."Fiscal Entity No.")


$\textbf{🟠\ High\ Severity\ —\ AppSource}$

The new namespaced NL layer repeatedly adds unaffixed members to standard objects, for example field(11400; "Cash Journal"; ...) in src/Layers/NL/BaseApp/Local/Foundation/AuditCodes/SourceCodeSetupNL.TableExt.al:14, field("Fiscal Entity No."; ...) in src/Layers/NL/BaseApp/Local/Foundation/Company/CompanyInformationNL.PageExt.al:13, and action("Post Code Updates") in src/Layers/NL/BaseApp/Local/RoleCenters/AdministratorMainRCNL.PageExt.al:15. A two-level namespace only replaces the owned object affix; fields, controls, and actions added to base objects still need the registered member affix.

Knowledge:

microsoft/knowledge/appsource/two-level-namespace-replaces-object-affix-not-extension-member-affix.md

microsoft/knowledge/appsource/object-affixes-prevent-collisions.md

_{👍 useful · ❤️ especially valuable · 👎 wrong - reply with why · AL review agent v1.12.4}

github-actions · 2026-07-20T11:47:56Z

$\textbf{🟡\ Medium\ Severity\ —\ Breaking\ Changes}$

The public IntegrationEvent OnBeforeUpdateVendorBankAccounts, moved from the base Vendor codeunit into the new VendorNL.TableExt.al, changed signature from (var IsHandled: Boolean; var VendorBankAccount: Record "Vendor Bank Account") to (var Vendor: Record Vendor; var IsHandled: Boolean; var VendorBankAccount: Record "Vendor Bank Account"). Any existing subscriber compiled against the old signature will fail to bind. Restore the original event signature and introduce a separate event if subscribers also need the Vendor record. Note: the underlying impact is a real breaking change to a public extensibility point, but this is emitted as an advisory agent finding capped at minor severity because no loaded knowledge file currently covers event-publisher signature changes directly — it should be promoted to a knowledge-backed rule.

Line mapping was unavailable, so this was posted as an issue comment.

_{👍 useful · ❤️ especially valuable · 👎 wrong - reply with why · AL review agent v1.12.4}

github-actions · 2026-07-20T11:47:58Z

+            end;
+        }
+        field(11000002; "Currency Euro"; Code[10])
+        {


$\textbf{🟠\ High\ Severity\ —\ Error\ Handling}$

The Currency Euro OnValidate path dead-ends with a plain Error even though the code already knows the immediate

Recommendation:

when Local Currency is Euro, Currency Euro must be cleared. Raise an ErrorInfo with a Fix-it action so the user can apply the correction in one step instead of dismissing a blocking dialog.

Knowledge:

microsoft/knowledge/error-handling/prefer-errorinfo-for-actionable-errors.md

_{👍 useful · ❤️ especially valuable · 👎 wrong - reply with why · AL review agent v1.12.4}

github-actions · 2026-07-20T11:47:59Z

+    }
+
+    var
+        TransactionModeAccountTypeErrorQst: Label '%1 can only be filled in when %2 %3 is equal to Customer or Vendor.', Comment = '%1 - Field Caption of the field causing the error, %2 - Field Caption of the Account Type field, %3 - Field Caption of the Bal. Account Type field';


$\textbf{🟡\ Medium\ Severity\ —\ Error\ Handling}$

The validation error text on GenJournalLineNL.TableExt.al says Transaction Mode Code is allowed only for Customer or Vendor, but the surrounding TableRelation and lookup logic also accept Employee. Update the message so users who hit this error are told the real set of supported account types.

Suggested change

TransactionModeAccountTypeErrorQst: Label '%1 can only be filled in when %2 %3 is equal to Customer or Vendor.', Comment = '%1 - Field Caption of the field causing the error, %2 - Field Caption of the Account Type field, %3 - Field Caption of the Bal. Account Type field';

TransactionModeAccountTypeErrorQst: Label '%1 can only be filled in when %2 %3 is equal to Customer, Vendor, or Employee.', Comment = '%1 - Field Caption of the field causing the error, %2 - Field Caption of the Account Type field, %3 - Field Caption of the Bal. Account Type field';

_{👍 useful · ❤️ especially valuable · 👎 wrong - reply with why · AL review agent v1.12.4}

github-actions · 2026-07-20T11:48:01Z

$\textbf{🟠\ High\ Severity\ —\ Privacy}$

The touched G/L Account field Omit Default Descr. in Jnl. is a Normal stored field but still has no DataClassification property, both before and after this PR's relocation of the field within the object. AS0016 requires field-level classification on every Normal field; leaving this field unclassified keeps its privacy posture undefined.

Knowledge:

microsoft/knowledge/privacy/table-level-data-classification-cascades.md

Line mapping was unavailable, so this was posted as an issue comment.

_{👍 useful · ❤️ especially valuable · 👎 wrong - reply with why · AL review agent v1.12.4}

github-actions · 2026-07-20T11:48:04Z

+
+tableextension 11382 "G/L Entry NL" extends "G/L Entry"
+{
+    fields


$\textbf{🟠\ High\ Severity\ —\ Upgrade}$

src/Layers/NL/BaseApp/Local/Finance/GeneralLedger/Ledger/GLEntryNL.TableExt.al reintroduces field 11301 Open (moved verbatim, including its InitValue = true, from the base G/L Entry table which had the same field removed in this diff) as a new tableextension field, but the NL upgrade work added in this PR only registers upgrade tags for Local SEPA Instr. Priority and Elec. Tax Declaration Setup. Moving a field's ownership from a base table to a table extension does not automatically carry over existing row values; without a tagged upgrade step, pre-upgrade G/L Entry rows will read back the Boolean default false instead of their real historical Open value.

Knowledge:

microsoft/knowledge/upgrade/initvalue-does-not-update-existing-rows.md

_{👍 useful · ❤️ especially valuable · 👎 wrong - reply with why · AL review agent v1.12.4}

…ests

ventselartur · 2026-07-23T09:10:19Z

        UnapplyVendorLedgerEntryAmount('');  // Currency as blank.
    end;

-    [Test]


Why do you remove the test?

All-E already reverted this change, so after sybc these tests and their earlier fix will dissappear anyway. I did it earlier myself to identify if this is the last issue with tests or I have smething else.

https://github.com/microsoft/BCApps into bugs/Extract-NL-customization-to-object-extensions-v2

github-actions · 2026-07-24T08:11:17Z

+/// Defines country-specific VAT registration number formats for validation and duplicate checking.
+/// Provides pattern matching capabilities and business rules enforcement for VAT numbers across different jurisdictions.
+/// </summary>
+codeunit 13381 "VAT Reg. No. Format NL"


$\textbf{🟡\ Medium\ Severity\ —\ Breaking\ Changes}$

Three new NL localization codeunits are declared without Access = Internal;, unlike every one of the other ~19 new NL event-subscriber codeunits added in this PR (e.g. "Employee NL", "Sales Header NL", "Customer NL"), which all consistently set it. "VAT Reg. No. Format NL" (codeunit 13381), "VAT Statement Line NL" (codeunit 11366), and "Company Information NL" (codeunit 11361) contain only local event-subscriber procedures with no intended external entry point, so leaving them without an access modifier makes them default-public and reachable from other extensions, turning implementation detail into an accidental contract.

Suggested fix (apply manually — could not be anchored as a one-click suggestion):

codeunit 13381 "VAT Reg. No. Format NL" { Access = Internal;

Knowledge:

microsoft/knowledge/breaking-changes/choose-access-modifiers-deliberately.md

_{👍 useful · ❤️ especially valuable · 👎 wrong - reply with why · AL review agent v1.15.4}

github-actions · 2026-07-24T08:11:18Z

+// ------------------------------------------------------------------------------------------------
+namespace Microsoft.Finance.VAT.Reporting;
+
+codeunit 11366 "VAT Statement Line NL"


$\textbf{🟡\ Medium\ Severity\ —\ Breaking\ Changes}$

Same missing Access = Internal; inconsistency as codeunit 13381 "VAT Reg. No. Format NL": this codeunit contains only a local event-subscriber procedure and should be internal like its sibling NL codeunits.

Suggested fix (apply manually — could not be anchored as a one-click suggestion):

codeunit 11366 "VAT Statement Line NL" { Access = Internal;

Knowledge:

microsoft/knowledge/breaking-changes/choose-access-modifiers-deliberately.md

_{👍 useful · ❤️ especially valuable · 👎 wrong - reply with why · AL review agent v1.15.4}

github-actions · 2026-07-24T08:11:20Z

+
+using Microsoft.Utilities;
+
+codeunit 11361 "Company Information NL"


$\textbf{🟡\ Medium\ Severity\ —\ Breaking\ Changes}$

Same missing Access = Internal; inconsistency as codeunit 13381 "VAT Reg. No. Format NL": this codeunit contains only a local event-subscriber procedure and should be internal like its sibling NL codeunits.

Suggested fix (apply manually — could not be anchored as a one-click suggestion):

codeunit 11361 "Company Information NL" { Access = Internal;

Knowledge:

microsoft/knowledge/breaking-changes/choose-access-modifiers-deliberately.md

_{👍 useful · ❤️ especially valuable · 👎 wrong - reply with why · AL review agent v1.15.4}

Extract NL customization to object extensions (TrMode)

a84e4c6

Alexander-Ya requested review from a team June 30, 2026 13:39

github-actions Bot reviewed Jun 30, 2026

View reviewed changes

Comment thread src/Layers/NL/BaseApp/Local/Finance/GeneralLedger/Journal/GenJournalLineNL.TableExt.al

github-actions Bot reviewed Jun 30, 2026

View reviewed changes

Comment thread src/Layers/NL/BaseApp/Local/Finance/GeneralLedger/Setup/GeneralLedgerSetupNL.TableExt.al Outdated

github-actions Bot reviewed Jun 30, 2026

View reviewed changes

Comment thread src/Layers/NL/BaseApp/Local/Foundation/Address/PostCodeRange.Table.al

github-actions Bot reviewed Jun 30, 2026

View reviewed changes

Comment thread src/Layers/NL/BaseApp/Local/Purchases/Vendor/VendorNL.TableExt.al

github-actions Bot reviewed Jun 30, 2026

View reviewed changes

Comment thread src/Layers/NL/BaseApp/Local/Purchases/Vendor/VendorNL.TableExt.al

github-actions Bot reviewed Jun 30, 2026

View reviewed changes

Comment thread src/Layers/NL/BaseApp/Local/Sales/Customer/CustomerNL.TableExt.al

aholstrup1 mentioned this pull request Jun 30, 2026

Trigger full PR build on changes under src/Layers #8939

Merged

4 tasks

Alexander-Ya added 3 commits June 30, 2026 18:59

Update solution

3d45ece

Update solution

17505f8

Remove delocalized objects

3cf3e95

github-actions Bot reviewed Jun 30, 2026

View reviewed changes

Comment thread src/Layers/NL/BaseApp/Local/HumanResources/Employee/EmployeeCardNL.PageExt.al

github-actions Bot reviewed Jun 30, 2026

View reviewed changes

Comment thread src/Layers/NL/BaseApp/Local/Finance/VAT/Registration/VATRegNoFormatNL.Codeunit.al

github-actions Bot reviewed Jun 30, 2026

View reviewed changes

Comment thread src/Layers/NL/BaseApp/Local/Foundation/Company/COmpanyInformationNL.Codeunit.al

github-actions Bot reviewed Jun 30, 2026

View reviewed changes

Comment thread src/Layers/NL/BaseApp/Local/Finance/GeneralLedger/Ledger/GLEntryNL.TableExt.al

Alexander-Ya added 4 commits June 30, 2026 22:38

Add missing changes

884ee8d

Merge branch 'main' of https://github.com/microsoft/BCApps into bugs/…

58cc153

…Extract-NL-customization-to-object-extensions-v2

Update 2 Role Centers

a2c7897

Merge branch 'main' of https://github.com/microsoft/BCApps into bugs/…

ca18779

…Extract-NL-customization-to-object-extensions-v2

This comment was marked as resolved.

Sign in to view

github-actions Bot reviewed Jul 1, 2026

View reviewed changes

Comment thread src/Layers/NL/BaseApp/Local/Sales/Customer/CustomerNL.TableExt.al

github-actions Bot reviewed Jul 1, 2026

View reviewed changes

Comment thread src/Layers/NL/BaseApp/Local/Sales/Customer/CustomerNL.TableExt.al

Add missing changes

b85a07a

github-actions Bot added this to the Version 29.0 milestone Jul 1, 2026

github-actions Bot reviewed Jul 17, 2026

View reviewed changes

Merge branch 'main' of https://github.com/microsoft/BCApps into bugs/…

bb7d296

…Extract-NL-customization-to-object-extensions-v2

github-actions Bot reviewed Jul 18, 2026

View reviewed changes

Alexander-Ya added 3 commits July 20, 2026 11:09

Merge branch 'main' of https://github.com/microsoft/BCApps into bugs/…

d7e89c0

…Extract-NL-customization-to-object-extensions-v2

Fix NL tests and remove delocalized objects

06ca2fb

Sync changes and resolve merge issues

3dfb410

github-actions Bot reviewed Jul 20, 2026

View reviewed changes

Copilot started work on behalf of Alexander-Ya July 21, 2026 11:00 View session

Fix NL UncategorizedTests: remove accidentally re-added failing WHT t…

25cb4c4

…ests

Copilot finished work on behalf of Alexander-Ya July 21, 2026 11:10

github-actions Bot added needs-approval Workflow runs require maintainer approval to start AL: Apps (W1) Add-on apps for W1 and removed needs-approval Workflow runs require maintainer approval to start labels Jul 21, 2026

ventselartur reviewed Jul 23, 2026

View reviewed changes

Copilot started work on behalf of Alexander-Ya July 23, 2026 20:33 View session

Restore NL data classification override

57f0941

Copilot finished work on behalf of Alexander-Ya July 23, 2026 20:40

Alexander-Ya added 2 commits July 24, 2026 10:01

Resolve merge conflicts

75e28dc

Merge branch 'bugs/Extract-NL-customization-to-object-extensions-v2' of

e6651d6

https://github.com/microsoft/BCApps into bugs/Extract-NL-customization-to-object-extensions-v2

github-actions Bot removed the AL: Apps (W1) Add-on apps for W1 label Jul 24, 2026

github-actions Bot reviewed Jul 24, 2026

View reviewed changes

	procedure UpdateElecTaxDeclCategoryCode()
	internal procedure UpdateElecTaxDeclCategoryCode()

	TransactionModeAccountTypeErrorQst: Label '%1 can only be filled in when %2 %3 is equal to Customer or Vendor.', Comment = '%1 - Field Caption of the field causing the error, %2 - Field Caption of the Account Type field, %3 - Field Caption of the Bal. Account Type field';
	TransactionModeAccountTypeErrorQst: Label '%1 can only be filled in when %2 %3 is equal to Customer, Vendor, or Employee.', Comment = '%1 - Field Caption of the field causing the error, %2 - Field Caption of the Account Type field, %3 - Field Caption of the Bal. Account Type field';


		using Microsoft.Utilities;

		codeunit 11361 "Company Information NL"

Uh oh!

Conversation

Alexander-Ya commented Jun 30, 2026 • edited by github-actions Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What & why

Linked work

How I validated this

Risk & compatibility

Uh oh!

Alexander-Ya commented Jun 30, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jun 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Copilot PR Review

Findings by domain

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

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

This comment was marked as resolved.

Uh oh!

Uh oh!

github-actions Bot Jul 17, 2026

Choose a reason for hiding this comment

The new EventSubscriber OnBeforePostCustomerEntry on Codeunit::"Sales-Post Prepayments" is declared with only two parameters, `(var GenJnlLine: Record "Gen.

Uh oh!

This comment was marked as off-topic.

Uh oh!

github-actions Bot Jul 18, 2026

Choose a reason for hiding this comment

VATStatementLineNL.TableExt widens UpdateElecTaxDeclCategoryCode from a local procedure in the original full-object NL override to a public tableextension procedure, but it is only invoked from the companion VATStatementLineNL.Codeunit.al within the same app.

Uh oh!

github-actions Bot Jul 18, 2026

Choose a reason for hiding this comment

CustomerNL.TableExt widens UpdateCustomerBankAccounts from a local procedure in the original full-object NL override to a published, non-local procedure marked only with [Scope('OnPrem')], even though every call site remains inside the same tableextension.

Uh oh!

github-actions Bot Jul 20, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Jul 20, 2026

Uh oh!

github-actions Bot Jul 20, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jul 20, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Jul 20, 2026

Uh oh!

github-actions Bot Jul 20, 2026

Choose a reason for hiding this comment

Uh oh!

ventselartur Jul 23, 2026

Choose a reason for hiding this comment

Uh oh!

Alexander-Ya Jul 23, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jul 24, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jul 24, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jul 24, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Alexander-Ya commented Jun 30, 2026 •

edited by github-actions Bot

Loading

github-actions Bot commented Jun 30, 2026 •

edited

Loading