Skip to content

feat: update on issue 1378 - #1391

Merged
afurious merged 1 commit into
Crowdfunding-DApp:mainfrom
williamsokperez-arch:feature/factory
Aug 29, 2026
Merged

feat: update on issue 1378#1391
afurious merged 1 commit into
Crowdfunding-DApp:mainfrom
williamsokperez-arch:feature/factory

Conversation

@williamsokperez-arch

Copy link
Copy Markdown

closes #1378

Description

Related Issues

Closes #

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • CI / Infrastructure

Required CI Checks

The following status checks must pass before this PR can be merged:

Frontend CI (frontend_ci.ymlci-frontend job):

  • TypeScript type-check (pnpm --filter @stellar-raise/frontend typecheck)
  • Vitest test suite (pnpm --filter @stellar-raise/frontend test)
  • ESLint across all workspaces (pnpm lint)
  • Prettier format check (pnpm format:check)

Rust CI (rust_ci.ymlcheck job):

  • cargo fmt --all -- --check
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo build --release --target wasm32-unknown-unknown
  • cargo test --workspace

Checklist

  • My branch is based off develop, not main
  • I have run cargo fmt --all and the code is properly formatted
  • I have run cargo clippy --all-targets -- -D warnings with no warnings
  • I have run cargo test and all tests pass
  • I have run pnpm typecheck with no type errors
  • I have run pnpm test and all frontend tests pass
  • I have run pnpm lint with no ESLint errors
  • I have run pnpm format:check with no formatting violations
  • I have added tests for any new functionality
  • All public functions have /// doc comments
  • I have updated CHANGELOG.md if applicable
  • My commit messages follow the conventional commits format

Screenshots / Logs (if applicable)

Additional Notes

@afurious
afurious merged commit 6b85e1d into Crowdfunding-DApp:main Aug 29, 2026
afurious added a commit that referenced this pull request Sep 3, 2026
The check job of Rust CI fails at the formatting step on an unformatted
assert_eq! in the factory tests (merged via #1391), and would fail the
next step on a clippy::useless_conversion in the crowdfund tests (merged
via #1392) where std::string::String::from wraps an already-owned String.

- factory: reflow assert_eq! per rustfmt
- crowdfund: drop the redundant String::from around "a".repeat(n)
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.

[Audit #76] Factory's Flagged CampaignStatus has zero on-chain enforcement on the underlying contract

3 participants