-
Notifications
You must be signed in to change notification settings - Fork 31
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 non-zero check of nextTokenID.Id for genesis (backport #1276) #1282
fix: add non-zero check of nextTokenID.Id for genesis (backport #1276) #1282
Conversation
* fix: add non-zero check of nextTokenID.Id for genesis * chore: add testcase * chore: update changelog * chore: move validation logic to validate function * chore: add testcase (cherry picked from commit 47a5e9f) # Conflicts: # CHANGELOG.md
Cherry-pick of 47a5e9f has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## release/v0.48.x #1282 +/- ##
===================================================
- Coverage 69.78% 69.78% -0.01%
===================================================
Files 646 646
Lines 67503 67506 +3
===================================================
Hits 47108 47108
- Misses 18209 18212 +3
Partials 2186 2186
|
Co-authored-by: 170210 <85928898+170210@users.noreply.github.com>
Description
tokenID
in GenesisMotivation and context
How has this been tested?
Screenshots (if appropriate):
Checklist:
CHANGELOG.md
client/docs/swagger-ui/swagger.yaml
This is an automatic backport of pull request #1276 done by [Mergify](https://mergify.com).