Skip to content

Conversation

@Dzeranov
Copy link
Contributor

@Dzeranov Dzeranov commented Apr 15, 2025

Issue tracking

Part of #3084

Context behind the change

  • webhook module refactoring
  • New unit tests

How has this been tested?

  • unit tests
  • manually deployed and tested affected endpoints

Release plan

N/A

Potential risks; What to monitor; Rollback plan

N/A

@Dzeranov Dzeranov self-assigned this Apr 15, 2025
@vercel
Copy link

vercel bot commented Apr 15, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
human-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 24, 2025 6:22am
human-dashboard-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 24, 2025 6:22am
staking-dashboard ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 24, 2025 6:22am
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
faucet-frontend ⬜️ Ignored (Inspect) Visit Preview Apr 24, 2025 6:22am
faucet-server ⬜️ Ignored (Inspect) Visit Preview Apr 24, 2025 6:22am

@Dzeranov Dzeranov moved this to In Progress in Core-tech - 2025 Apr 15, 2025
@Dzeranov Dzeranov requested a review from dnechay April 22, 2025 12:20
Copy link
Collaborator

@dnechay dnechay left a comment

Choose a reason for hiding this comment

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

LGTM

Some minor comments for cleanup

@dnechay dnechay merged commit b03557b into develop Apr 24, 2025
13 checks passed
@github-project-automation github-project-automation bot moved this from In Review to Done in Core-tech - 2025 Apr 24, 2025
@dnechay dnechay deleted the dzeranov/3084-webhook branch April 24, 2025 08:32
@dnechay dnechay mentioned this pull request Apr 24, 2025
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants