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 treasury account on reset scripts #910

Merged
merged 1 commit into from
Oct 12, 2023
Merged

Fix treasury account on reset scripts #910

merged 1 commit into from
Oct 12, 2023

Conversation

rokn
Copy link
Contributor

@rokn rokn commented Oct 12, 2023

Detailed description:
Fix treasury account and wrong association which lead to missing token ids

Special notes for your reviewer:

Checklist

  • Documentation added
  • Tests updated

@codecov
Copy link

codecov bot commented Oct 12, 2023

Codecov Report

Merging #910 (5e222b9) into main (9eb5524) will not change coverage.
The diff coverage is n/a.

❗ Current head 5e222b9 differs from pull request most recent head aebb369. Consider uploading reports for the commit aebb369 to get more accurate results

@@           Coverage Diff           @@
##             main     #910   +/-   ##
=======================================
  Coverage   64.29%   64.29%           
=======================================
  Files          86       86           
  Lines        6716     6716           
=======================================
  Hits         4318     4318           
  Misses       2184     2184           
  Partials      214      214           
Flag Coverage Δ
unittests 64.29% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Coiling-Dragon
Coiling-Dragon previously approved these changes Oct 12, 2023
Signed-off-by: Antonio Mindov <antonio.mindov@limechain.tech>
@rokn rokn merged commit b96b334 into main Oct 12, 2023
4 of 5 checks passed
@rokn rokn deleted the fix/reset-scripts branch October 12, 2023 14:12
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.

2 participants