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
Budget command supports moving budgets between wallets. #8262
Conversation
Can you have a look at the acceptance test for budgets and add in this new functionality? Or are you happy with unit tests? https://github.com/juju/juju/blob/develop/acceptancetests/assess_budget.py |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still LGTM, thanks for updating these tests!
:build: |
!!build!! |
@howbazaar - @anastasiamac suggested I should get your review on this |
@howbazaar, |
|
Status: merge request accepted. Url: http://ci.jujucharms.com/job/github-merge-juju |
…ving-wallets Budget command supports moving budgets between wallets. This is a forward of #8262 ## Description of change This change enables users to use the `juju budget` command to move a budget to a different wallet (as well as update the budget's limit). This change was requested by beta users. ## QA steps Running the `juju budget` command with a parameter referencing an existing wallet should move the budget to the new wallet. ## Documentation changes The change augments the `juju budget` command.
Let the Docs team (me) know if something needs changing at the online documentation level. |
Description of change
This change enables users to use the
juju budget
command to move a budget to a different wallet (as well as update the budget's limit).This change was requested by beta users.
QA steps
Running the
juju budget
command with a parameter referencing an existing wallet should move the budget to the new wallet.Documentation changes
The change augments the
juju budget
command.