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(cmd/rofl): Add initial support for ROFL app management #265

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

kostko
Copy link
Member

@kostko kostko commented Jul 9, 2024

No description provided.

Copy link

netlify bot commented Jul 9, 2024

Deploy Preview for oasisprotocol-cli canceled.

Name Link
🔨 Latest commit 53eddfc
🔍 Latest deploy log https://app.netlify.com/sites/oasisprotocol-cli/deploys/66c49a4c13a94100080dfb45

@kostko kostko linked an issue Jul 9, 2024 that may be closed by this pull request
@kostko kostko force-pushed the kostko/feature/rofl branch 5 times, most recently from 3971f57 to c72d1ce Compare July 15, 2024 09:55
@kostko kostko requested a review from matevz July 16, 2024 14:27
Copy link
Member

@matevz matevz left a comment

Choose a reason for hiding this comment

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

Please also add a brief example to the docs (add rofl.md).

cmd/rofl.go Outdated
}

roflCreateCmd = &cobra.Command{
Use: "create <policy.json>",
Copy link
Member

Choose a reason for hiding this comment

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

Please add an example policy to the docs.

cmd/rofl.go Outdated Show resolved Hide resolved
@kostko kostko force-pushed the kostko/feature/rofl branch 4 times, most recently from c1b9808 to fc183c8 Compare July 20, 2024 12:21
@matevz
Copy link
Member

matevz commented Aug 19, 2024

How can a user know what's the staking threshold to register a ROFL app? This should probably go somewhere here right?

@kostko
Copy link
Member Author

kostko commented Aug 19, 2024

Not there as that one is for network-wide stuff? The ROFL subcommand could mention it somewhere, e.g. when trying to create a new ROFL app and not having enough funds?

@kostko kostko force-pushed the kostko/feature/rofl branch from fc183c8 to 68591e4 Compare August 20, 2024 13:25
@kostko kostko force-pushed the kostko/feature/rofl branch from 68591e4 to 53eddfc Compare August 20, 2024 13:29
@kostko kostko merged commit 51f1b63 into master Aug 22, 2024
4 checks passed
@kostko kostko deleted the kostko/feature/rofl branch August 22, 2024 14:15
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.

Add support for ROFL app operations
2 participants