Update ConnectivityAppDefinitions.Codeunit.al - #11263
Conversation
Summary: Added Bankfeed to 10 more countries in ConnectivityApps: BG, CA, FR, GR, IT, MT, NL, NO, UA, ZA Work Item(s): Fixes
|
Could not find linked issues in the pull request description. Please make sure the pull request description contains a line that contains 'Fixes #' followed by the issue number being fixed. Use that pattern for every issue you want to link. |
Good Sense Reviewer - Round 1Recommendation: Request ChangesWhat this PR doesThe change adds Softera Bankfeed availability for more countries and adds matching description translations. The implementation updates the same metadata strings that Problem-solution fitFit: Partial The requested rollout is clear, and the edit is in the right registration method. The solution is incomplete because several changed values cannot be evaluated by the existing enums. SuggestionsS1 (🔴 High): Use valid country and localization values Risk assessment and necessityRisk: This touches metadata that is loaded for the Connectivity Apps list. If the invalid values remain, users can hit a runtime failure before the list is built; there is no posting, ledger, or event-subscriber risk. Necessity: The expansion is reasonable because it makes an existing banking app available in more countries. The scope is narrow, but it must use supported country/region and localization values before merge.
|
Summary:
Added Bankfeed to 10 more countries in ConnectivityApps: BG, CA, FR, GR, IT, MT, NL, NO, UA, ZA
Work Item(s):
Fixes #