Skip to content

Fix default chain id#16

Closed
philipsu522 wants to merge 1 commit intomasterfrom
fix-default-chain-name
Closed

Fix default chain id#16
philipsu522 wants to merge 1 commit intomasterfrom
fix-default-chain-name

Conversation

@philipsu522
Copy link
Contributor

App Name is "sei", chain is usually "sei-chain"

@LCyson
Copy link
Contributor

LCyson commented May 25, 2022

What will this change on the cmd side?

@philipsu522
Copy link
Contributor Author

it defaults to "sei-chain". Thinking about this more let's discard and keep it the same as other chains (default chain created is "sei" not "sei-chain"

masih pushed a commit that referenced this pull request Sep 29, 2025
masih pushed a commit that referenced this pull request Oct 1, 2025
* checkpoint

* set config

* more fixes

* Add logging

* more fixes

* fixes

* more fixes

* more logging

* fixes

* Add logs

* fix outerloop

* outerloop fixes 2

* fix proto

* more logging

* Add proposal event

* fix data block hash

* Add more logs

* add proposal block for prevote

* checkpoint

* checkpoint

* implement trycreateproposal

* lower logging, impl tx req

* Add metrics

* register labels

* fix typo metric

* cleanup

* continue gossiping block parts

* uncomment

* Add logging

* fix nil proposal

* more logs, gossip txn false

* set flag to true

* Fix added logic

* reduce logs

* less logging

* less loggign, gossip-tx=flase

* gossip-tx-key true

* code cleanup, turn off recheck

* fix config typo

* remove logs

* cleanup

* more cleanup

* Add try proposal span

* tryCreate once

* fix trycreate

* cleanup, fix metrics

* more fixes/cleanup

* address comments

* protogen
masih pushed a commit that referenced this pull request Oct 9, 2025
[Refactor] Fix benchmark build tags and rlog -> changelog
@masih masih deleted the fix-default-chain-name branch October 31, 2025 15:48
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