Skip to content

[Shopify] Disambiguate Shopify timestamp field captions and tooltips#9253

Open
onbuyuka wants to merge 1 commit into
mainfrom
bugs/641693-shopify-timestamp-captions
Open

[Shopify] Disambiguate Shopify timestamp field captions and tooltips#9253
onbuyuka wants to merge 1 commit into
mainfrom
bugs/641693-shopify-timestamp-captions

Conversation

@onbuyuka

@onbuyuka onbuyuka commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

What & why

Shopify-sourced timestamp fields (createdAt, updatedAt, etc.) were stored with generic captions like "Created At" / "Updated At", which are easily confused with the BC platform's SystemCreatedAt / SystemModifiedAt auditing fields (visible in search, personalization, and extension development).

This PR extends the disambiguation pattern already used in Shpfy Orders To Import to the rest of the connector:

  • Adds a (Shopify) suffix to 20 timestamp field captions across 11 tables (Company, Customer, Product, Variant, Inventory Item, Order Header, Order Fulfillment, Fulfillment Order Header, Refund Header, Payment Transaction, Order Transaction).
  • Fixes the CreatedAt (missing space) caption bug on Shpfy Variant.
  • Standardizes the related page tooltips to mention "in Shopify" for a clear data source, fixes the "was update" typo on the refund tooltips, and corrects the misleading "…is processed" wording on the Order Transaction "Created At" tooltip.

Text/caption-only change — no schema, logic, or API changes.

Linked work

Fixes AB#641693

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

  • Compiled the Shopify Connector app locally (AL compiler) — 0 errors, no new warnings.
  • Verified via search that all 20 target captions now carry the (Shopify) suffix and that no timestamp tooltip in scope is left without "in Shopify".
  • No tests added: this is a caption/tooltip text-only change with no behavioral impact; there is no test coverage for on-screen caption/tooltip strings.

Risk & compatibility

  • Display-only. Field IDs, field names, data, and API/OData behavior are unchanged, so there is no upgrade or breaking-change impact.
  • Non-English translations for the affected captions/tooltips will show as outdated until the translation pipeline regenerates them — expected and handled by the normal localization flow.

Shopify-sourced timestamp fields (createdAt/updatedAt/etc.) were stored
with generic captions like "Created At"/"Updated At", easily confused
with BC platform's SystemCreatedAt/SystemModifiedAt auditing fields.

- Add "(Shopify)" suffix to 20 timestamp field captions across 11 tables,
  matching the existing pattern in Shpfy Orders To Import.
- Fix the "CreatedAt" (missing space) caption bug on Shpfy Variant.
- Standardize the related page tooltips to mention "in Shopify", fix the
  "was update" typo on the refund tooltips, and correct the misleading
  "processed" wording on the Order Transaction "Created At" tooltip.

Fixes AB#641693

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@onbuyuka onbuyuka requested a review from a team July 8, 2026 13:11
@github-actions github-actions Bot added the AL: Apps (W1) Add-on apps for W1 label Jul 8, 2026
@github-actions github-actions Bot added this to the Version 29.0 milestone Jul 8, 2026
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Copilot PR Review

Iteration 1 · Outcome: completed

Knowledge source: https://github.com/microsoft/BCQuality@822cae1b2771ac25f665f73369f69093bd4fd630

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

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

Findings produced by the Copilot CLI agent against BCQuality at 822cae1b2771ac25f665f73369f69093bd4fd630. Reply 👎 on any inline comment to flag false positives.

@onbuyuka onbuyuka enabled auto-merge July 8, 2026 13:18
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.

2 participants