Skip to content

Audit Trails API #164

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.

Sign up for GitHub

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

Jump to bottom
Merged
itsyaasir merged 21 commits into from
Dec 24, 2025
Merged

Audit Trails API #164

itsyaasir merged 21 commits into from
Dec 24, 2025
+2,342 −74

Conversation

@itsyaasir
Copy link
Contributor

@itsyaasir itsyaasir commented Dec 17, 2025

Description of change

Links to any relevant issues

Type of change

  • Bug fix (a non-breaking change which fixes an issue)
  • Enhancement (a non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Fix

How the change has been tested

Change checklist

  • I have followed the contribution guidelines for this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that new and existing unit tests pass locally with my changes
  • I have updated the CHANGELOG.md, if my changes are significant enough

@itsyaasir itsyaasir self-assigned this Dec 17, 2025
chrisgitiota
chrisgitiota reviewed Dec 19, 2025
View reviewed changes
chrisgitiota
chrisgitiota reviewed Dec 19, 2025
View reviewed changes
chrisgitiota
chrisgitiota reviewed Dec 19, 2025
View reviewed changes
chrisgitiota
chrisgitiota reviewed Dec 19, 2025
View reviewed changes
chrisgitiota
chrisgitiota reviewed Dec 19, 2025
View reviewed changes
chrisgitiota
chrisgitiota reviewed Dec 19, 2025
View reviewed changes
chrisgitiota
chrisgitiota reviewed Dec 19, 2025
View reviewed changes
chrisgitiota and others added 9 commits December 21, 2025 16:30
@chrisgitiota
First version of permission
d1d8dc5 
* First implementation of the permission module based on a `Permission` enum
* Unit tests for the `Permission` enum
* Renamed `AuditTrail::permissions` to `AuditTrail::roles`
* Renamed all modules and type-names from plural to singular name
  * audit_trails -> audit_trail
  * permissions -> permission
  * capabilities -> capability
@chrisgitiota
First version of initial roles config and Admin role Capability creation
632d1e7 
Unit tests are still buggy and will be fixed with the next commit.
@chrisgitiota
Add missing tests for create AT
f5b6b2a
@chrisgitiota
Fixes for the create AT tests
395a68f
@chrisgitiota
New test test_create_metadata_admin_role() in create_tests.move
713c494
@chrisgitiota
Fix dprint issues
6cb397b
@itsyaasir
refactor: Rename audit_trails module to main and update locking confi…
bc512d8 
…guration structure
@chrisgitiota
New test for the role-based access control delegation workflow
ed79a98
@chrisgitiota
Add access control check for update_metadata()
09ae22b
@itsyaasir itsyaasir marked this pull request as ready for review December 23, 2025 07:56
@itsyaasir itsyaasir changed the title [WIP] : Audit Trails API Audit Trails API Dec 23, 2025
chrisgitiota and others added 9 commits December 23, 2025 11:58
@chrisgitiota @itsyaasir
Update audit-trail-move/sources/permission.move
75c8487 
Rename MetaDataUpdate to MetadataUpdate

Co-authored-by: Yasir <yasir@shariff.dev>
@chrisgitiota
Implementation for the issued_capabilities whitelist management plus …
60f5562 
…corresponding tests
@chrisgitiota
Merge branch 'feat/audit-trails-api-roles-management' of github.com:i…
7938050 
…otaledger/notarization into feat/audit-trails-api-roles-management
@chrisgitiota
Globally rename 'MetaData' to 'Metadata'
520eda0
@chrisgitiota
Rename 'create_tests' to 'create_audit_trail_tests'
e5a71a1
@chrisgitiota
Rename all Permissions variants and creator functions according to th…
cfb8c84 
…e verb-subject format
@chrisgitiota
Merge branch 'feat/audit-trails-api' into feat/audit-trails-api-roles…
ffe4871 
…-management

# Conflicts:
#	audit-trail-move/sources/locking.move
#	audit-trails-move/sources/audit_trails.move
@chrisgitiota
Rewrite of all LockingConfig creating function calls in Move tests
600ae26
@chrisgitiota
Add access control check for AuditTrail.update_metadata() and `delete…
c029e29 
…_record_lock`
@itsyaasir
Merge pull request #173 from iotaledger/feat/audit-trails-api-roles-m…
09f1a96 
…anagement

Feat/audit trails api roles management
@itsyaasir itsyaasir merged commit e89679a into feat/audit-trails-dev Dec 24, 2025
11 checks passed
@itsyaasir itsyaasir deleted the feat/audit-trails-api branch December 24, 2025 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chrisgitiota chrisgitiota chrisgitiota left review comments

Assignees

@itsyaasir itsyaasir

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@itsyaasir @chrisgitiota