-
Notifications
You must be signed in to change notification settings - Fork 47
Updating Stable with Main #328
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…d Jest (#311) * feat: add security testing framework and configuration for Cypress and Jest * feat: update Cypress configuration and commands for improved security testing * add dependency
* refactor: update sender fee calculation to use human-readable cap - Changed maxFeeCapInBaseUnits to derive from a human-readable max fee cap for clarity. - Improved code readability by separating human-readable and base unit calculations. - Ensured consistency in fee calculations for local transfers. * feat: replace Noblocks logo with holiday-themed SVG * refactor: simplify max fee cap calculation in calculateSenderFee function --------- Co-authored-by: Chibuotu Amadi <chibuotu@gmail.com> Co-authored-by: Francis Ocholi <5raan6@gmail.com>
…related components (#315) * feat: remove references to Hedera Mainnet from network filtering and related components * feat: remove Hedera Mainnet reference from gateway contract address mapping
…e unused scroll effect in TransactionStatus (#318)
* feat: add Ethereum support with RPC URLs and token definitions * feat: update token addresses for USDC, USDT, and cNGN in FALLBACK_TOKENS --------- Co-authored-by: chibie <chibuotu@gmail.com>
* feat: enhance Brevo chat widget integration * fix: correct Brevo chat widget group ID property name and update integration logic * Update app/components/AppLayout.tsx Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * Update app/components/AppLayout.tsx Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * Update .env.example Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> --------- Co-authored-by: chibie <chibuotu@gmail.com> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
- Changed footer rocket illustration to a new Santa illustration. - Adjusted margin in WalletView component for better spacing. - Simplified conditional logic in network filtering.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Description
This pull request introduces several improvements and updates across the application, focusing on seasonal branding, Brevo chat widget enhancements, network support adjustments, and critical security documentation. The most notable changes include a comprehensive security analysis report, seasonal UI updates for the holidays, and improved configuration and logic for the Brevo chat widget. Additionally, there are refinements to the supported blockchain networks and minor usability enhancements.
Security & Documentation
SECURITY_REPORT.md), highlighting critical gaps in testing infrastructure, identifying high-risk areas (such as authentication, transaction processing, and input validation), and providing a phased strategy for implementing security tests. This report serves as a roadmap for immediate and future security improvements.UI/UX & Seasonal Branding
Navbar, and changing the footer illustration to a Santa image. [1] [2] [3]Brevo Chat Widget Improvements
NEXT_PUBLIC_BREVO_CONVERSATIONS_IDandNEXT_PUBLIC_BREVO_CONVERSATIONS_GROUP_IDin.env.example.Blockchain Network Support Adjustments
Refined network filtering logic throughout the application to only exclude "Celo" for non-injected (smart) wallets, instead of excluding both "Celo" and "Hedera Mainnet". This affects several components, ensuring more consistent and accurate network availability. [1] [2] [3] [4] [5] [6]
Added Ethereum Mainnet (
mainnet) to the list of supported chains in the Privy configuration, broadening wallet compatibility. [1] [2]Usability Enhancements
These changes collectively enhance the application’s security posture, user experience, and seasonal appeal, while also improving configuration flexibility and blockchain compatibility.
References
Testing
Checklist
mainBy submitting a PR, I agree to Paycrest's Contributor Code of Conduct and Contribution Guide.