Skip to content

Commit c407d6f

Browse files
committed
Enhance route handling by adding warnings for invalid policyID in multiple accounting routes
1 parent 1e1c630 commit c407d6f

1 file changed

Lines changed: 48 additions & 9 deletions

File tree

src/ROUTES.ts

Lines changed: 48 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -3881,36 +3881,75 @@ const ROUTES = {
38813881
},
38823882
POLICY_ACCOUNTING_CERTINIA_EXISTING_CONNECTIONS: {
38833883
route: 'workspaces/:policyID/accounting/certinia/existing-connections',
3884-
getRoute: (policyID: string) => `workspaces/${policyID}/accounting/certinia/existing-connections` as const,
3884+
getRoute: (policyID: string | undefined) => {
3885+
if (!policyID) {
3886+
Log.warn('Invalid policyID is used to build the POLICY_ACCOUNTING_CERTINIA_EXISTING_CONNECTIONS route');
3887+
}
3888+
return `workspaces/${policyID}/accounting/certinia/existing-connections` as const;
3889+
},
38853890
},
38863891
POLICY_ACCOUNTING_CERTINIA_IMPORT: {
38873892
route: 'workspaces/:policyID/accounting/certinia/import',
3888-
getRoute: (policyID: string | undefined) => `workspaces/${policyID}/accounting/certinia/import` as const,
3893+
getRoute: (policyID: string | undefined) => {
3894+
if (!policyID) {
3895+
Log.warn('Invalid policyID is used to build the POLICY_ACCOUNTING_CERTINIA_IMPORT route');
3896+
}
3897+
return `workspaces/${policyID}/accounting/certinia/import` as const;
3898+
},
38893899
},
38903900
POLICY_ACCOUNTING_CERTINIA_DIMENSION_MAPPING: {
38913901
route: 'workspaces/:policyID/accounting/certinia/import/dimension/:dimension',
3892-
getRoute: (policyID: string | undefined, dimension: 'dimension1' | 'dimension2' | 'dimension3' | 'dimension4') =>
3893-
`workspaces/${policyID}/accounting/certinia/import/dimension/${dimension}` as const,
3902+
getRoute: (policyID: string | undefined, dimension: 'dimension1' | 'dimension2' | 'dimension3' | 'dimension4') => {
3903+
if (!policyID) {
3904+
Log.warn('Invalid policyID is used to build the POLICY_ACCOUNTING_CERTINIA_DIMENSION_MAPPING route');
3905+
}
3906+
return `workspaces/${policyID}/accounting/certinia/import/dimension/${dimension}` as const;
3907+
},
38943908
},
38953909
POLICY_ACCOUNTING_CERTINIA_EXPORT: {
38963910
route: 'workspaces/:policyID/accounting/certinia/export',
3897-
getRoute: (policyID: string | undefined) => `workspaces/${policyID}/accounting/certinia/export` as const,
3911+
getRoute: (policyID: string | undefined) => {
3912+
if (!policyID) {
3913+
Log.warn('Invalid policyID is used to build the POLICY_ACCOUNTING_CERTINIA_EXPORT route');
3914+
}
3915+
return `workspaces/${policyID}/accounting/certinia/export` as const;
3916+
},
38983917
},
38993918
POLICY_ACCOUNTING_CERTINIA_PREFERRED_EXPORTER: {
39003919
route: 'workspaces/:policyID/accounting/certinia/export/preferred-exporter',
3901-
getRoute: (policyID: string | undefined) => `workspaces/${policyID}/accounting/certinia/export/preferred-exporter` as const,
3920+
getRoute: (policyID: string | undefined) => {
3921+
if (!policyID) {
3922+
Log.warn('Invalid policyID is used to build the POLICY_ACCOUNTING_CERTINIA_PREFERRED_EXPORTER route');
3923+
}
3924+
return `workspaces/${policyID}/accounting/certinia/export/preferred-exporter` as const;
3925+
},
39023926
},
39033927
POLICY_ACCOUNTING_CERTINIA_EXPORT_STATUS: {
39043928
route: 'workspaces/:policyID/accounting/certinia/export/status',
3905-
getRoute: (policyID: string | undefined) => `workspaces/${policyID}/accounting/certinia/export/status` as const,
3929+
getRoute: (policyID: string | undefined) => {
3930+
if (!policyID) {
3931+
Log.warn('Invalid policyID is used to build the POLICY_ACCOUNTING_CERTINIA_EXPORT_STATUS route');
3932+
}
3933+
return `workspaces/${policyID}/accounting/certinia/export/status` as const;
3934+
},
39063935
},
39073936
POLICY_ACCOUNTING_CERTINIA_EXPORT_DATE: {
39083937
route: 'workspaces/:policyID/accounting/certinia/export/date',
3909-
getRoute: (policyID: string | undefined) => `workspaces/${policyID}/accounting/certinia/export/date` as const,
3938+
getRoute: (policyID: string | undefined) => {
3939+
if (!policyID) {
3940+
Log.warn('Invalid policyID is used to build the POLICY_ACCOUNTING_CERTINIA_EXPORT_DATE route');
3941+
}
3942+
return `workspaces/${policyID}/accounting/certinia/export/date` as const;
3943+
},
39103944
},
39113945
POLICY_ACCOUNTING_CERTINIA_DEFAULT_VENDOR: {
39123946
route: 'workspaces/:policyID/accounting/certinia/export/default-vendor',
3913-
getRoute: (policyID: string | undefined) => `workspaces/${policyID}/accounting/certinia/export/default-vendor` as const,
3947+
getRoute: (policyID: string | undefined) => {
3948+
if (!policyID) {
3949+
Log.warn('Invalid policyID is used to build the POLICY_ACCOUNTING_CERTINIA_DEFAULT_VENDOR route');
3950+
}
3951+
return `workspaces/${policyID}/accounting/certinia/export/default-vendor` as const;
3952+
},
39143953
},
39153954
POLICY_ACCOUNTING_CERTINIA_ADVANCED: {
39163955
route: 'workspaces/:policyID/accounting/certinia/advanced',

0 commit comments

Comments
 (0)