Merged
Conversation
…rvices - Added deployment logs for TokenRegistry and TestERC20. - Introduced new deployment script for OauthCore. - Created Docker Compose configuration for relayer services, including PostgreSQL setup. - Added Dockerfiles for IMAP and SMTP services. - Updated Rust code to include new environment variables and improve error handling in DKIM oracle. - Enhanced ABI definitions for various contracts, including ERC20, ERC721, and token registry. These changes facilitate the deployment and management of the relayer services and improve the overall architecture.
- Removed the 'selector' parameter from the computeSignedMsg and revokeDKIMPublicKeyHash functions in ECDSAOwnedDKIMRegistry contract. - Updated ABI definitions to reflect the changes in function signatures. - Enhanced deployment logs with updated contract addresses for various components, including TokenRegistry and Oauth extensions. - Improved error handling and logging in the ChainClient implementation. These changes streamline the contract's functionality and improve the clarity of deployment logs.
… variable adjustments - Added Node version requirement and usage instructions to README. - Updated .env.example to reflect changes in RPC explorer URL, database connection string, and web server address.
…racts and running the relayer
…d Rust for improved contract compilation and runtime environment
…mlined service builds
- Added PostgreSQL deployment and service configuration in db.yml. - Updated ingress and managed certificate configurations to use the new domain and namespace. - Refactored relayer configuration, including environment variables and service definitions, to align with the new namespace and improve service management.
… relayer - Changed namespace from email-wallet to email-wallet-v11 in Kubernetes configurations. - Updated ingress and managed certificate to reflect new domain. - Enhanced error handling in relayer code for file operations and JSON parsing. - Added a new function to read email templates with error handling.
- Updated cronjob to restart the relayer deployment in the new namespace. - Removed obsolete db.yml file containing PostgreSQL configurations. - Refactored relayer configuration, including environment variables and service definitions, to improve clarity and maintainability.
Feat/relayer docker
benceharomi
requested changes
Mar 25, 2025
Member
benceharomi
left a comment
benceharomi
left a comment
There was a problem hiding this comment.
In my opinion defaulting to local deployment instead of base deployment for chain id, rpc, etc. would make more sense.
Co-authored-by: benceharomi <56651250+benceharomi@users.noreply.github.com>
Co-authored-by: benceharomi <56651250+benceharomi@users.noreply.github.com>
Co-authored-by: benceharomi <56651250+benceharomi@users.noreply.github.com>
Co-authored-by: benceharomi <56651250+benceharomi@users.noreply.github.com>
…ubernetes manifests
… into refactor/v1.1
Co-authored-by: benceharomi <56651250+benceharomi@users.noreply.github.com>
… into refactor/v1.1
benceharomi
reviewed
Mar 27, 2025
benceharomi
reviewed
Mar 27, 2025
benceharomi
reviewed
Mar 27, 2025
benceharomi
reviewed
Mar 27, 2025
Co-authored-by: benceharomi <56651250+benceharomi@users.noreply.github.com>
benceharomi
requested changes
Apr 8, 2025
Member
benceharomi
left a comment
benceharomi
left a comment
There was a problem hiding this comment.
I added my final comments, if these are resolved it is good to go!
benceharomi
requested changes
Apr 8, 2025
Member
benceharomi
left a comment
benceharomi
left a comment
There was a problem hiding this comment.
One other thing: can you please also run cargo fmt and cargo clippy to format and lint the code?
Co-authored-by: benceharomi <56651250+benceharomi@users.noreply.github.com>
Contributor
Author
ran cargo fmt. But Clippy has too many warnings (150+ and no auto fix) to resolve for this PR. |
benceharomi
approved these changes
Apr 9, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add deployment guide, ICP refactoring, and relayer enhancements
Changes
Created a new
DESIGN.mdfile that contains the original project overview, features, and technical detailsTransformed the main
README.mdinto a detailed deployment guide with step-by-step instructionsAdded instructions for:
Refactored Internet Computer Protocol (ICP) integration:
Benefits
Testing
The deployment guide has been tested on Base Sepolia network and works as expected. The ICP refactoring and relayer enhancements have been tested in development environment.