Microsoft/bcquality
1
0
Fork 0
mirror of https://github.com/microsoft/BCQuality.git synced 2026-10-06 23:26:55 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

knowledge: improve review precision from BCApps PR 10990-neg-3926331607-source-faithful-20261005 feedback

Browse source
This commit is contained in:
github-actions[bot] 2026-10-05 08:01:36 +00:00
parent ac249ba4c9
commit 0d728298ef
1 changed files with 36 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

36
microsoft/knowledge/upgrade/contoso-demo-data-seeding-does-not-require-upgrade-codeunit.md Normal file
View file

@ -0,0 +1,36 @@
---
bc-version: [all]
domain: upgrade
keywords: [contoso-demo-data-module, demo-tool, demo-data, upgrade-codeunit, preview, false-positive]
technologies: [al]
countries: [w1]
application-area: [all]
---
# Contoso demo-data seeding does not need an upgrade codeunit
## Description
Codeunits that `implements "Contoso Demo Data Module"` (the Contoso demo-data-tool
interface, e.g. `CreateSetupData`, `CreateMasterData`, `CreateTransactionalData`,
`CreateHistoricalData`) are only ever invoked on demand by the demo data tool when a
user explicitly chooses to generate or refresh Contoso demo data. They are not part of
the extension's install/upgrade lifecycle: the platform never calls them automatically,
and no already-provisioned tenant silently receives their records on extension
upgrade — the user must rerun the demo tool to get new or changed Contoso records, for
every change in that data, not just this one. Adding new seeding steps (for example a
new reference/rate table population) inside `CreateMasterData`/`CreateSetupData` of
such a module is consistent with every other Contoso demo-data change and does not
leave real tenant data stale. The `upgrade-codeunit-subtype` and
`install-code-does-not-run-on-version-upgrade` rules govern code that must run against
real, already-provisioned tenant data; they do not apply to Contoso demo/preview data
generation.
## Best Practice
Do not require `Subtype = Upgrade` wiring, `OnUpgradePerCompany`/`OnUpgradePerDatabase`
entry points, or an upgrade codeunit for new seeding logic added to a codeunit that
implements `"Contoso Demo Data Module"` and is only reached through
`CreateSetupData`/`CreateMasterData`/`CreateTransactionalData`/`CreateHistoricalData`.
Flag missing upgrade wiring only when the changed code also runs for real tenants
outside the demo tool's on-demand generation flow.