Backport #10990: [29.x] Add Expense VAT settings to Contoso demo tool - #10994
Backport #10990: [29.x] Add Expense VAT settings to Contoso demo tool#10994Alexander Yakunin (Alexander-Ya) wants to merge 3 commits into
Conversation
Good Sense Reviewer - Round 1Recommendation: Request ChangesWhat this PR doesThe change wires Expense VAT rate creation into the Contoso demo module for the 29.x branch. This should let demo data create VAT product posting groups, VAT posting setup, and default VAT settings for Expense Agent categories. The new call is not enough to create a complete setup. The VAT rate code depends on the Expense Agent setup already having a default VAT business posting group. The existing setup helper fills that default and records that VAT rates were applied, but the direct codeunit call bypasses both parts. Problem-solution fitFit: Partial The change adds VAT rates to demo master data, but it does not apply the full setup state that the VAT flow needs. A demo company can still be left without the default VAT business posting group used by agent-created VAT specifications. SuggestionsS1 (🔴 High): Use the setup helper for VAT defaults Risk assessment and necessityRisk: This affects Contoso Expense Agent demo setup and VAT posting setup. If the default VAT business group is missing or the rates are created against a blank group, demo expenses can fail VAT specification insertion or use incomplete VAT setup. Necessity: The change is needed so demo data includes the VAT settings used by Expense Agent. The scope is right, but it needs to use the setup path that creates a complete configuration.
|
…nto backport/releases/29.x/10990/20260903114017
Good Sense Reviewer - Round 2Recommendation: Request ChangesWhat this PR doesThe update changes Expense Agent demo setup to use a country-aware setup path, adds VAT-rate G/L accounts for demo data, and sets the default VAT business posting group before creating VAT rates. It also carries more expense fields into report lines and adds country-specific per-diem demo data. The previous blank VAT business posting group issue is fixed in the demo flow. However, the shared VAT-rate creation code now depends on demo G/L accounts that are not created by the setup default path, and the demo flow does not mark VAT defaults as applied after creating them. Status of previous suggestions
New observations (commits since round 1)S2 (🔴 High): Default VAT setup can miss purchase account S3 (🟠 Moderate): Mark VAT defaults as applied Risk assessment and necessityRisk: This touches VAT Posting Setup, default VAT groups, and report VAT specification amounts. If the VAT setup is incomplete, expense VAT posting can fail or use incomplete account setup. Necessity: Adding VAT setup to Expense Agent demo data is needed so the demo company can create usable VAT specifications. The scope is appropriate, but the default setup path must stay complete and the setup state must match the inserted VAT data.
|
Backport of #10990 to
releases/29.x.Original change: Add Expense VAT settings to the Contoso demo tool.
Bug: AB#648890