Skip to content

feat(contract): add get_config getter (closes #173) - #201

Closed
kevindurant735rocket-creator wants to merge 2 commits into
DepRadarOrg:mainfrom
kevindurant735rocket-creator:fix/get-config-173
Closed

kevindurant735rocket-creator wants to merge 2 commits into
DepRadarOrg:mainfrom
kevindurant735rocket-creator:fix/get-config-173

Conversation

@kevindurant735rocket-creator

Copy link
Copy Markdown

Closes #173.

Adds Config struct and get_config() returning the router's configured limits:
REVIEW_WINDOW_SECS, MAX_BOUNTY_DURATION_SECS, MAX_CLAIMS_PER_BOUNTY,
MAX_REF_LEN. So off-chain clients do not hard-code them.

  • cargo fmt, cargo clippy -- -D warnings, cargo test --workspace (22 tests) pass.
  • Added test_get_config_matches_constants.
  • PROGRESS.md entry added.

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.

[contract] Add a get_config getter

2 participants