Skip to content

Add CI pipelines, SDK examples, and harden admin authorization - #554

Merged
ayshadogo merged 1 commit into
Dfunder:mainfrom
kilodesodiq-arch:fix/issue-527-observability-and-admin-auth
Jul 30, 2026
Merged

Add CI pipelines, SDK examples, and harden admin authorization#554
ayshadogo merged 1 commit into
Dfunder:mainfrom
kilodesodiq-arch:fix/issue-527-observability-and-admin-auth

Conversation

@kilodesodiq-arch

Copy link
Copy Markdown
Contributor

Summary

Implemented CI infrastructure, developer examples, and security hardening for admin authorization.

Changes

  • Created CI workflow (ci.yml) for Rust tests, TypeScript tests, clippy, and formatting on PRs/pushes
  • Added structured observability logging to donation_verifier with span context for debugging failures
  • Created TypeScript SDK examples (campaign-workflow.ts) covering create campaign, donate, and withdraw flows
  • Fixed admin authorization in platform_config: removed incorrect env.current_contract_address().require_auth() from set_fee_bps
  • Normalized admin.require_auth() as the consistent auth pattern across all admin functions

Closes #527
Closes #528
Closes #529
Closes #530

- Create .github/workflows/ci.yml with Rust tests, TypeScript tests, clippy, and formatting checks
- Add observability logging to donation_verifier with structured tracing events
- Create TypeScript SDK examples for campaign creation, donation, and withdrawal workflows
- Fix admin authorization in platform_config: remove incorrect contract self-auth in set_fee_bps
- Normalize admin.require_auth() as the sole auth pattern across admin functions

Closes Dfunder#527
Closes Dfunder#528
Closes Dfunder#529
Closes Dfunder#530
@drips-wave

drips-wave Bot commented Jul 29, 2026

Copy link
Copy Markdown

@kilodesodiq-arch Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@ayshadogo ayshadogo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ayshadogo
ayshadogo merged commit 1784646 into Dfunder:main Jul 30, 2026
0 of 4 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

2 participants