Skip to content

chore: add privy config prod#856

Merged
pedroanastacio merged 2 commits intomainfrom
p2/chore/add-privy-config-prod
Mar 12, 2026
Merged

chore: add privy config prod#856
pedroanastacio merged 2 commits intomainfrom
p2/chore/add-privy-config-prod

Conversation

@pedroanastacio
Copy link
Copy Markdown
Contributor

Description

  • Add Privy env vars

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 implemented error handling for all actions/requests and verified how they will be displayed in the UI (or there was no error handling needed).
  • I mentioned the PR link in the task

Copy link
Copy Markdown
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 has 1 commit(s) not in source. Consider syncing before merge.

📦 Changes Included

1 PR(s) being promoted:

PR Author Issues
#855 FIX: add privy env vars @pedroanastacio -

✅ No issues detected

Copy link
Copy Markdown
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 0b7a1d8 into main Mar 12, 2026
1 of 2 checks passed
@pedroanastacio pedroanastacio deleted the p2/chore/add-privy-config-prod branch March 12, 2026 19:43
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.

2 participants