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

Add reservation commands #21

Merged
merged 11 commits into from
Nov 2, 2022
Merged

Add reservation commands #21

merged 11 commits into from
Nov 2, 2022

Conversation

the-masthead
Copy link
Contributor

  • Add project change props command
  • Add transfer project ownership command
  • Allow setting owner at project creation
  • Add reservation create, delete, list commands
  • Move shared flags to separate help group
  • Print tx error stack trace if DEBUG is set

Copy link
Contributor

@the-flagship the-flagship left a comment

Choose a reason for hiding this comment

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

One small comment

@@ -7,7 +7,7 @@ import { getAll, getContract, getProvider, normalizeHex, normalizeRecords } from
export default class ReservationList extends BlockchainCommand {
static description = "Lists node reservations by a project.";
static examples = ["<%= config.bin %> <%= command.id %>"];
static usage = "<%= command.id %> [--topology] [--operator ID] [--skip N] [--size N]";
static usage = "<%= command.id %> [--skip N] [--size N]";
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing --page

@the-masthead the-masthead merged commit 4929894 into main Nov 2, 2022
@the-masthead the-masthead deleted the reservation branch November 2, 2022 22:51
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