rust/hyperswitch
1
0
Fork 0
mirror of https://github.com/juspay/hyperswitch.git synced 2025-11-02 21:07:58 +08:00
Code Issues Packages Projects Releases Wiki Activity

chore(roles): remove redundant variant from PermissionGroup (#6985)

Browse Source
This commit is contained in:
Kashif
2025-02-06 19:13:13 +05:30
committed by GitHub
parent f9a4713a60
commit 775dcc5a4e
5 changed files with 9 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
crates/common_enums/src/enums.rs
View File

@ -2936,8 +2936,6 @@ pub enum PermissionGroup {
ReconReportsManage,
ReconOpsView,
ReconOpsManage,
// TODO: To be deprecated, make sure DB is migrated before removing
ReconOps,
}
#[derive(Clone, Debug, serde::Serialize, PartialEq, Eq, Hash, strum::EnumIter)]

6
crates/router/src/services/authorization/info.rs
View File

@ -40,10 +40,10 @@ fn get_group_description(group: PermissionGroup) -> &'static str {
PermissionGroup::MerchantDetailsView | PermissionGroup::AccountView => "View Merchant Details",
PermissionGroup::MerchantDetailsManage | PermissionGroup::AccountManage => "Create, modify and delete Merchant Details like api keys, webhooks, etc",
PermissionGroup::OrganizationManage => "Manage organization level tasks like create new Merchant accounts, Organization level roles, etc",
PermissionGroup::ReconReportsView => "View and access reconciliation reports and analytics",
PermissionGroup::ReconReportsView => "View reconciliation reports and analytics",
PermissionGroup::ReconReportsManage => "Manage reconciliation reports",
PermissionGroup::ReconOpsView => "View and access reconciliation operations",
PermissionGroup::ReconOpsManage | PermissionGroup::ReconOps => "Manage reconciliation operations",
PermissionGroup::ReconOpsView => "View and access all reconciliation operations including reports and analytics",
PermissionGroup::ReconOpsManage => "Manage all reconciliation operations including reports and analytics",
}
}

5
crates/router/src/services/authorization/permission_groups.rs
View File

@ -33,7 +33,6 @@ impl PermissionGroupExt for PermissionGroup {
| Self::OrganizationManage
| Self::AccountManage
| Self::ReconOpsManage
| Self::ReconOps
| Self::ReconReportsManage => PermissionScope::Write,
}
}
@ -50,7 +49,7 @@ impl PermissionGroupExt for PermissionGroup {
| Self::MerchantDetailsManage
| Self::AccountView
| Self::AccountManage => ParentGroup::Account,
Self::ReconOpsView | Self::ReconOpsManage | Self::ReconOps => ParentGroup::ReconOps,
Self::ReconOpsView | Self::ReconOpsManage => ParentGroup::ReconOps,
Self::ReconReportsView | Self::ReconReportsManage => ParentGroup::ReconReports,
}
}
@ -86,7 +85,7 @@ impl PermissionGroupExt for PermissionGroup {
}
Self::ReconOpsView => vec![Self::ReconOpsView],
Self::ReconOpsManage | Self::ReconOps => vec![Self::ReconOpsView, Self::ReconOpsManage],
Self::ReconOpsManage => vec![Self::ReconOpsView, Self::ReconOpsManage],
Self::ReconReportsView => vec![Self::ReconReportsView],
Self::ReconReportsManage => vec![Self::ReconReportsView, Self::ReconReportsManage],

1
migrations/2025-01-03-104019_migrate_permission_group_for_recon/down.sql Normal file
View File

@ -0,0 +1 @@
SELECT 1;

3
migrations/2025-01-03-104019_migrate_permission_group_for_recon/up.sql Normal file
View File

@ -0,0 +1,3 @@
UPDATE roles
SET groups = array_replace(groups, 'recon_ops', 'recon_ops_manage')
WHERE 'recon_ops' = ANY(groups);