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.

Sign up for GitHub

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

Jump to bottom

cli: allow to set default arguments from config #47

Closed
tzemanovic opened this issue Jan 10, 2022 · 4 comments · Fixed by #2466
Closed

cli: allow to set default arguments from config #47

tzemanovic opened this issue Jan 10, 2022 · 4 comments · Fixed by #2466
Assignees
@Fraccaman
Labels
cli

Comments

@tzemanovic
Copy link
Member

tzemanovic commented Jan 10, 2022 •
edited by sync-by-unito bot
Loading

We should replace e.g. the hard-coded default --ledger-address with the one from the config file

┆Issue is synchronized with this Asana task by Unito
@tzemanovic tzemanovic added the cli label Jan 10, 2022
@tzemanovic
Copy link
Member Author

This will simplify the explicit --ledger-address and --node arguments added to e2e tests in https://github.com/anoma/anoma/pull/799

@sync-by-unito sync-by-unito bot closed this as completed Feb 3, 2022
@juped juped reopened this Feb 3, 2022
@tzemanovic tzemanovic mentioned this issue Jul 7, 2022
Closed
@tzemanovic tzemanovic transferred this issue from anoma/anoma-archive Jul 7, 2022
@cwgoes
Copy link
Contributor

cwgoes commented Jan 12, 2023

@Fraccaman do you know the status of this?

@tzemanovic
Copy link
Member Author

to be done

@tzemanovic tzemanovic mentioned this issue Jan 24, 2024
Closed
@gavinly
Copy link
Contributor

gavinly commented Jan 25, 2024

would be great to have this

@tzemanovic tzemanovic mentioned this issue Jan 29, 2024
Merged
2 tasks
phy-chain pushed a commit to phy-chain/namada that referenced this issue Mar 1, 2024
@memasdeligeorgakis
anoma#47 Staking, Governance, Public Goods Funding - initialising vie…
7982395 
…ws and themes (anoma#53)

* Manually copied over the changes from feat/47_staking_gov_pgf, as there
  had been a big refactor and this could not be merged automatically
* Had to add containers as we have `Appcomponents__ContentContainer`
  which is not `display: flex` and in the account views things would break
  in the current form, if this was to be changed to flex. Should be
  refactored though
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Fraccaman Fraccaman

Labels
cli
Projects
No open projects
Namada-Old
Status: Tested in Devnet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

client: use node address from config if not specified anoma/namada
5 participants
@juped @tzemanovic @Fraccaman @gavinly @cwgoes