You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@fineract.apache.org by "Satish11012007 (via GitHub)" <gi...@apache.org> on 2026/02/28 17:13:18 UTC
[PR] FINERACT-1430: Added social status and restricted high profile client access [fineract]
Satish11012007 opened a new pull request, #5561:
URL: https://github.com/apache/fineract/pull/5561
## Summary
This PR introduces access restriction for high profile clients using a new social status field.
### Changes included:
- Added Liquibase changelog to create `m_client.social_status_cv_id`
- Added foreign key constraint from `m_client.social_status_cv_id` to `m_code_value.id`
- Inserted new permission entry `CAN_VIEW_HIGH_PROFILE_CLIENT` into `m_permission`
- Included the new `FINERACT-1430` changelog in `db.changelog-master.xml`
- Added `socialStatus` field in `Client` entity mapped to `social_status_cv_id`
- Enforced permission validation when reading a client with social status set
- Added `PermissionConstants.CAN_VIEW_HIGH_PROFILE_CLIENT` constant
---
## Behavior
- If `social_status_cv_id` is **null**, the client can be viewed normally.
- If `social_status_cv_id` is **not null**, the system validates the required permission:
```java
context.authenticatedUser()
.validateHasPermissionTo(
PermissionConstants.CAN_VIEW_HIGH_PROFILE_CLIENT
);
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@fineract.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FINERACT-1430: Added social status and restricted high profile client access [fineract]
Posted by "IOhacker (via GitHub)" <gi...@apache.org>.
IOhacker commented on PR #5561:
URL: https://github.com/apache/fineract/pull/5561#issuecomment-3979492261
@Aman-Mittal in finance we use a different wording and different approach either for KYC, AM/FT but never we use "social status" wording because at least in the uses cases that I have been working either on commercial or development banks use that for refering to a person and it is consider discrimination.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@fineract.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FINERACT-1430: Added social status and restricted high profile client access [fineract]
Posted by "adamsaghy (via GitHub)" <gi...@apache.org>.
adamsaghy closed pull request #5561: FINERACT-1430: Added social status and restricted high profile client access
URL: https://github.com/apache/fineract/pull/5561
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@fineract.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FINERACT-1430: Added social status and restricted high profile client access [fineract]
Posted by "Aman-Mittal (via GitHub)" <gi...@apache.org>.
Aman-Mittal commented on PR #5561:
URL: https://github.com/apache/fineract/pull/5561#issuecomment-3979500858
> @Aman-Mittal in finance we use a different wording and different approach either for KYC, AML/FT but never we use "social status" wording because at least in the uses cases that I have been working either on commercial or development banks use that for refering to a person and it is consider discrimination. That is the reason that I don't approve the change submitted in the PR.
Thanks for replying.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@fineract.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FINERACT-1430: Added social status and restricted high profile client access [fineract]
Posted by "Aman-Mittal (via GitHub)" <gi...@apache.org>.
Aman-Mittal commented on PR #5561:
URL: https://github.com/apache/fineract/pull/5561#issuecomment-3979482320
> In financial terms, there is not "high/low social status" this seems to be a segregation feature.
>
> I don't approve this change
> In financial terms, there is not "high/low social status" this seems to be a segregation feature.
>
> I don't approve this change
Hi Victor, seems like this ticket is related to compliance usecase, There is no high and low social status but there can be High Net Worth clients (High Net Worth Indiviuals) or PEP (Politically Exposed Person). Maybe need to ask more to the reporter of this ticket for his usecase first.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@fineract.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FINERACT-1430: Added social status and restricted high profile client access [fineract]
Posted by "Aman-Mittal (via GitHub)" <gi...@apache.org>.
Aman-Mittal commented on PR #5561:
URL: https://github.com/apache/fineract/pull/5561#issuecomment-4178807339
> @Satish11012007 Please review the comments and address the issue!
@adamsaghy seems like this PR is being working on old abandoned ticket. And reporter didn't even replied to victor's comment in jira. I think we should close or mark this as draft.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@fineract.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FINERACT-1430: Added social status and restricted high profile client access [fineract]
Posted by "adamsaghy (via GitHub)" <gi...@apache.org>.
adamsaghy commented on PR #5561:
URL: https://github.com/apache/fineract/pull/5561#issuecomment-4022339209
@Satish11012007 Please rebase this PR
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@fineract.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FINERACT-1430: Added social status and restricted high profile client access [fineract]
Posted by "adamsaghy (via GitHub)" <gi...@apache.org>.
adamsaghy commented on PR #5561:
URL: https://github.com/apache/fineract/pull/5561#issuecomment-4054187835
@Satish11012007 Please rebase this PR and please configure GPG commit signature.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@fineract.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FINERACT-1430: Added social status and restricted high profile client access [fineract]
Posted by "adamsaghy (via GitHub)" <gi...@apache.org>.
adamsaghy commented on PR #5561:
URL: https://github.com/apache/fineract/pull/5561#issuecomment-4177710852
@Satish11012007 Please review the comments and address the issue!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@fineract.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FINERACT-1430: Added social status and restricted high profile client access [fineract]
Posted by "IOhacker (via GitHub)" <gi...@apache.org>.
IOhacker commented on PR #5561:
URL: https://github.com/apache/fineract/pull/5561#issuecomment-3979497187
I have added my comment in Jira too.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@fineract.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org