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

Allow setting verification cost to zero #939

Open
cameel opened this issue Nov 16, 2018 · 1 comment
Open

Allow setting verification cost to zero #939

cameel opened this issue Nov 16, 2018 · 1 comment
Assignees
@rwrzesien
Labels
bankster Bankster and related features feature sync with deployment There is a deployment change that has to be merged first verification use case Additional Verification use case

Comments

@cameel
Copy link
Contributor

cameel commented Nov 16, 2018

Currently Bankster assumes that verification always has non-zero cost. We need to allow setting it to zero (in that case no claim against provider's account should be created).

When done, set it to zero in deployment.

Remove the default from base settings. It should have no default (except maybe in development settings).
@cameel cameel added feature not in pivotal Not yet added as a task in any story in Pivotal verification use case Additional Verification use case bankster Bankster and related features labels Nov 16, 2018
@PaweuB PaweuB removed the not in pivotal Not yet added as a task in any story in Pivotal label Nov 26, 2018
@rwrzesien rwrzesien added the sync with deployment There is a deployment change that has to be merged first label Nov 26, 2018
@rwrzesien rwrzesien mentioned this issue Nov 26, 2018
Closed
@cameel
Copy link
Contributor Author

cameel commented Dec 7, 2018

Is this done?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rwrzesien rwrzesien

Labels
bankster Bankster and related features feature sync with deployment There is a deployment change that has to be merged first verification use case Additional Verification use case
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@cameel @rwrzesien @PaweuB