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

Fix Consensus #112

Merged
merged 5 commits into from
Dec 7, 2023
Merged

Fix Consensus #112

merged 5 commits into from
Dec 7, 2023

Conversation

taryune
Copy link
Member

@taryune taryune commented Nov 29, 2023

resolve #111

@taryune taryune changed the title [WIP] Fix Consensus Fix Consensus Dec 1, 2023
@taryune taryune requested a review from foxytanuki December 1, 2023 12:12
Copy link
Member

@foxytanuki foxytanuki left a comment

Choose a reason for hiding this comment

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

LGTM!
Confirmed it's working successfully with the steps below:

  1. docker compose build
  2. Run node1 and initialize node2 by docker compose up
  3. Run node2
  4. Send tx from node2
    1. myceld tx registry register-second-level-domain hoge cel 1 --from alice --keyring-backend test -y
    2. myceld tx registry update-dns-record hoge cel A 1.1.1.1 --from alice --keyring-backend test -y
    3. myceld tx registry register-top-level-domain aaa 1 --from alice --keyring-backend test -y

@taryune taryune merged commit 6f6578c into main Dec 7, 2023
3 checks passed
@taryune taryune deleted the fix/consensus branch December 7, 2023 12:27
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.

Consensus Error on Register Domain
3 participants