Skip to content

Conversation

@pedroanastacio
Copy link
Member

Description

  • Adds logs to get info about send tx error

Checklist

  • I reviewed my PR code before submitting
  • I ensured that the implementation is working correctly and did not impact other parts of the app
  • I mentioned the PR link in the task

Copy link
Member

@guimroque guimroque left a comment

Choose a reason for hiding this comment

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

🚀 Production Promotion Checklist

This PR is being merged to main. Please verify the items below:

🔒 Security

  • No secrets/credentials exposed in code
  • No staging/development URLs hardcoded
  • No debug flags enabled

✅ Quality

  • Build passed on staging
  • Tests passed on staging
  • Code was reviewed on staging

🔄 Integrity

✅ Branch main is synchronized

⚠️ Issues Found

🟡 Warnings (manual review)

  • Console statement: console.log( em packages/api/src/modules/transaction/controller.ts:481
  • Console statement: console.log( em packages/api/src/modules/transaction/services.ts:566

Copy link
Member

@guimroque guimroque left a comment

Choose a reason for hiding this comment

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

✅ Validation passed. PR approved for merge to main.

@pedroanastacio pedroanastacio merged commit e215327 into main Jan 21, 2026
1 check failed
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.

3 participants