Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Token summary not updated when using HTTP API #2049

Merged

Conversation

sameersubudhi
Copy link
Contributor

What was the problem?

This PR resolves #2048

How was it solved?

  • Update token info on blockchain state change

How was it tested?

Locally, against the mainnet with HTTP API enabled

Copy link

codecov bot commented Apr 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.89%. Comparing base (a138100) to head (8b15826).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##           development    #2049   +/-   ##
============================================
  Coverage        67.89%   67.89%           
============================================
  Files              303      303           
  Lines             5703     5703           
  Branches          1081     1081           
============================================
  Hits              3872     3872           
  Misses            1831     1831           

Copy link
Contributor

@nagdahimanshu nagdahimanshu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@sameersubudhi sameersubudhi merged commit 80aa704 into development Apr 10, 2024
11 checks passed
@sameersubudhi sameersubudhi deleted the 2048-token-summary-not-updated-when-using-http-api branch April 10, 2024 15:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Token summary not updated when using HTTP API
2 participants