Skip to content

Add Contributor Profile Completion Indicator #127

Merged
Abd-Standard merged 2 commits into
Core-Foundry:mainfrom
Rayhab2000:feat/contributor-profile-completeness
Jul 24, 2026
Merged

Add Contributor Profile Completion Indicator #127
Abd-Standard merged 2 commits into
Core-Foundry:mainfrom
Rayhab2000:feat/contributor-profile-completeness

Conversation

@Rayhab2000

Copy link
Copy Markdown
Contributor

What

Why

Closes #80

How

Testing

Breaking Changes


Checklist

General

  • Branch is up to date with main
  • PR title follows Conventional Commits format (e.g. feat: add reputation system)
  • PR description is complete with issue reference (Closes #<issue-number>)
  • One logical change per PR (no unrelated changes bundled in)

Testing

  • Tests added or updated to cover the changes
  • All tests pass (cargo test for contracts, npm run build for frontend)
  • No compiler warnings (cargo clippy -- -D warnings)

Documentation

  • Public API changes are documented with /// doc comments (Rust) or JSDoc (TypeScript)
  • CONTRIBUTING.md, README.md, or other docs updated if needed
  • CONTRACT_API.md updated if contract interface changed

Code Quality

  • Code follows the project style (cargo fmt / ESLint)
  • No any types introduced in TypeScript
  • No hardcoded secrets, keys, or environment-specific values

Smart Contract (if applicable)

  • require_auth() called on all state-changing operations
  • Events emitted for all state changes
  • Storage keys don't collide with existing keys
  • Deadline and input validation is complete
  • No unintended token transfer paths introduced

Frontend (if applicable)

  • Components are under 150 lines
  • UI is responsive and accessible
  • Screenshots or recordings included for visual changes

@Abd-Standard
Abd-Standard merged commit 783825e into Core-Foundry:main Jul 24, 2026
2 of 3 checks passed
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.

Add Contributor Profile Completion Indicator

2 participants