Skip to content

Contract storage#931

Closed
OluRemiFour wants to merge 10 commits into
Pulsefy:mainfrom
OluRemiFour:contract-storage
Closed

Contract storage#931
OluRemiFour wants to merge 10 commits into
Pulsefy:mainfrom
OluRemiFour:contract-storage

Conversation

@OluRemiFour

Copy link
Copy Markdown

Close: #863

Summary

I have successfully implemented and tested the authorization hardening for contributor metadata updates.

Changes completed:

Re-architected update_contributor enforcing explicit self-service authorization while maintaining state accurately.
Introduced update_contributor_admin routing through the Multisig ProposalAction::UpdateContributor pattern.
Integrated the ContributorUpdatedEvent emitting the actor modifying the state.
Wrote an extensive set of test cases verifying self-service vs admin-managed permissions natively into lib.rs.
All 43 unit tests now successfully pass. A walkthrough document has been created for your reference summarizing the changes.

Linked Issue

Closes #

Type of Change

  • feat
  • fix
  • docs
  • refactor
  • test
  • chore

Validation

  • Lint passed for affected area(s)
  • Tests passed for affected area(s)
  • Manual verification completed (if applicable)

Documentation

  • Documentation updated (or N/A with explanation)
  • Screenshots/videos attached for UI changes

Checklist

  • Branch name uses feat/, fix/, or docs/
  • Commit messages follow Conventional Commits
  • PR scope matches linked issue acceptance criteria

@OluRemiFour

Copy link
Copy Markdown
Author

Done, Close: #931

@Cedarich

Copy link
Copy Markdown
Contributor

@OluRemiFour fix workflow

@Cedarich

Copy link
Copy Markdown
Contributor

@OluRemiFour Fix CI

@Cedarich Cedarich closed this Jun 29, 2026
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.

Contracts: Contract storage usage introspection helpers

2 participants