Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[17.0][MIG] kyc #48

Merged
merged 37 commits into from
Jan 8, 2025
Merged

[17.0][MIG] kyc #48

merged 37 commits into from
Jan 8, 2025

Conversation

JasminSForgeFlow
Copy link
Contributor

@JasminSForgeFlow JasminSForgeFlow commented Dec 20, 2024

LoisRForgeFlow and others added 30 commits December 19, 2024 18:02
- Add Process logs
- Update changes for expired contact and contact about to expire
* Update partner with data entered in scan wizard.
* add a new security group to control access to logs.
* minor fixes in security.
* store last auto scan date.
* Improve auto scan logic.
* enchance extensibility (hook for domain of auto scan and
  centrilezed method to accept/reject transaction).
* rename "expire" fields, prepend with "kyc_" namespace.
- Remove restriction for government record for scan
- Add Group for delete documents
- Fix KYC scan issue with non admin user
…nments, hide messages of required documents for goverments
name_get in partner was completly overrriden which was leading to
changes in how partners are displayed in reports and SO (when
they include the address).
- do not change partner name.
- add expirations warnings in partner views.
- correct expiration computation.
Partners related to users are not accesible to everyone, users
with no rights might have issues accessing the contact screen or
the form view of a specific contact.
@LoisRForgeFlow
Copy link
Contributor

@JasminSForgeFlow Dependency merged, I restarted the CI and now it complains about a view definition.

@JasminSForgeFlow
Copy link
Contributor Author

@JasminSForgeFlow Dependency merged, I restarted the CI and now it complains about a view definition.

Its ready now,
Thanks

Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@LoisRForgeFlow LoisRForgeFlow merged commit 6b1d87c into 17.0 Jan 8, 2025
4 checks passed
@LoisRForgeFlow LoisRForgeFlow deleted the 17.0-mig-kyc branch January 8, 2025 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants