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

Problem: don't work on tendermint 0.35 #46

Merged
merged 1 commit into from
Feb 17, 2022

Conversation

yihuang
Copy link
Collaborator

@yihuang yihuang commented Feb 17, 2022

Closes: #45
Solution:

  • support new field names

Copy link
Contributor

@allthatjazzleo allthatjazzleo left a comment

Choose a reason for hiding this comment

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

LGTM

@yihuang yihuang marked this pull request as draft February 17, 2022 07:20
@yihuang
Copy link
Collaborator Author

yihuang commented Feb 17, 2022

still have some issues after these changes, need to investigate later.

@yihuang yihuang force-pushed the support-tm-0.35 branch 2 times, most recently from 36cd094 to 5c8b706 Compare February 17, 2022 09:21
@yihuang yihuang marked this pull request as ready for review February 17, 2022 09:22
Closes: crypto-com#45
Solution:
- support new field names
- remove self from peer list
- enable legacy p2p mode
@yihuang
Copy link
Collaborator Author

yihuang commented Feb 17, 2022

it works now.

@yihuang yihuang merged commit 1b6a152 into crypto-com:main Feb 17, 2022
@yihuang yihuang deleted the support-tm-0.35 branch February 17, 2022 09:32
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.

Problem: don't work on tendermint 0.35
2 participants