Skip to content
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

Fix: Add blockchains config #265

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

DancingAxolotl
Copy link
Contributor

Background

Adds a new config type to support custom base RPC URLs

Changes

  • Added UnsBlockchainsSource to SourceConfig

To Do

Additional Deployment and/or Rollback Steps

Code Review

This Pull Request was thoroughly reviewed and meets all Code Review Standards pertaining to:

  • Implementation - satisfied acceptance criteria
  • Communication - notified engineers/stakeholders about impactful changes
  • Error handling - handled, logged & alerted on errors
  • Documentation - wrote readable code & commits, documented unintuitive code
  • Testing - unit & E2E test coverage, tested in staging, DB migrations backwards compatible
  • Security - sanitized inputs, vetted dependencies, validated/secured API requests, no committed secrets
  • Performance - optimized expensive algorithms & database queries

Please select all applied standards relevant to this PR.

Confirmation

  • Assignee Confirmation - I (the author) have filled out the template above
  • Reviewer Confirmation - I (a/the reviewer) confirm 1) the author has filled out the template and checked the box that says Assignee Confirmation 2) I reviewed the code and selected all applicable items in the code review section.

@DancingAxolotl DancingAxolotl force-pushed the mike/PE-624/fix-config-params branch 2 times, most recently from 720c193 to ca973d6 Compare January 9, 2025 18:22
@DancingAxolotl DancingAxolotl force-pushed the mike/PE-624/fix-config-params branch from ca973d6 to f9d71fa Compare January 10, 2025 17:33
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.

1 participant