Skip to content

Add PocketPay Contracts issue approval readiness checklist #537

Description

@El-swaggerito

Description

Difficulty: Expert

Type: Documentation

Summary

Create an issue approval readiness checklist for PocketPay Contracts contributors.

Current Behaviour

Contributors may assume their work is complete once a PR is merged, even when tests, CI, documentation, or acceptance criteria are incomplete.

Expected Behaviour

The repo should provide a clear checklist that contributors and reviewers can use before considering an issue ready for evaluation.

Suggested Implementation

Add checklist documentation covering implementation completeness, tests, CI status, acceptance criteria, documentation, and known limitations.

Files or Areas Likely Affected

  • contracts/
  • src/
  • tests/
  • docs/
  • README.md
  • Makefile

Acceptance Criteria

  • Approval readiness checklist is added.
  • Checklist states that merged PRs are still subject to evaluation.
  • Testing expectations are included.
  • CI status expectations are included.
  • Acceptance criteria review is included.
  • README links to the checklist.

Additional Notes

This helps contributors evaluate their work before payment periods.

Metadata

Metadata

Assignees

Labels

GrantFox OSSIssue tracked for GrantFox OSS contributionMaybe RewardedIssue may be eligible for a GrantFox rewardOfficial Campaign | FWC26Official GrantFox campaign labelcontractSmart contract related workdeveloper-experienceDeveloper experience workdocumentationDocumentation workexpertExpert difficultysorobanSoroban smart contract work

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions