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

feat: fix startup bugs #265

Merged
merged 1 commit into from
Dec 4, 2023
Merged

feat: fix startup bugs #265

merged 1 commit into from
Dec 4, 2023

Conversation

CjS77
Copy link
Contributor

@CjS77 CjS77 commented Dec 4, 2023

  1. Wallet
  • There are 2 extra settings in the wallet config. clear_peer_db, clears the peer database when configuring the wallet.
    This should be true for at least the first run or until Peer DB does not respect TCP override tari#5998 is fixed.
  • interactive must be false for the first run, and can be set to true after that.
  • We only display the instructions for attaching to the docker wallet instance if interactive is true, and the wallet is active.
  1. Base node

Similarly to the wallet, there's an interactive setting for the base node now that must be false on the first run.

Note: A future PR should make these editable in the UI.

1. Wallet

There are 2 extra settings in the wallet config. clear_peer_db, clears
the peer database when configuring the wallet.
This should be true for at least the first run or until
tari-project/tari#5998 is fixed.

`interacive` must be false for the first run, and can be set to true
after that.
Adds a quick fix for the wallet for the non-interactive and peer DB
We only display the instructions for attaching to the docker wallet
instance if interactive is true, and the wallet is active.

bugs.

You can control whether the wallet is non-interactive (default. will run
on first startup), and whether the peer DB is deleted (true. shold let
2. Base node

Similarly to the wallet, there's an `interactive` setting for the base
node now that must be false on the first run.

A future PR should make these editable in the UI.
@CjS77 CjS77 merged commit 9924aba into main Dec 4, 2023
5 checks passed
@CjS77 CjS77 deleted the startup_settings branch December 4, 2023 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants