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 watch contract to update checksum address in stateStatusMap #377

Merged
merged 3 commits into from
May 10, 2023

Conversation

nikugogoi
Copy link
Collaborator

@nikugogoi nikugogoi commented May 10, 2023

Part of #375

  • Error is thrown on using a non checksum address for watching contract
    • Fixed by using converting to checksum address before updating stateStatusMap
  • Use non-null object type composer for object types to make corresponding fields in the generated GQL schema "required"

@nikugogoi nikugogoi self-assigned this May 10, 2023
@ashwinphatak ashwinphatak merged commit 2b4d98b into main May 10, 2023
@ashwinphatak ashwinphatak deleted the ng-fix-watch-contract branch May 10, 2023 13:19
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