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

BSIP77: Permissions and their forfeiture #277

Merged
merged 2 commits into from
May 29, 2020
Merged

BSIP77: Permissions and their forfeiture #277

merged 2 commits into from
May 29, 2020

Conversation

MichelSantos
Copy link
Contributor

Adding a permission to update the new initial collateral ratio (ICR) parameter in the same spirit as the discussion in #218

abitmore
abitmore reviewed May 28, 2020
View reviewed changes
Copy link
Member

@abitmore abitmore left a comment

Choose a reason for hiding this comment

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

Thanks.

BTW current implementation extended asset_publish_feed_operation so ICR can be updated by feed producers too.

@abitmore
Copy link
Member

abitmore commented May 29, 2020 •
edited
Loading

@MichelSantos please add yourself to the author list. Thanks.

Update: done for you.

@abitmore abitmore merged commit 197fda3 into bitshares:master May 29, 2020
@MichelSantos MichelSantos mentioned this pull request May 30, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@abitmore abitmore abitmore approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MichelSantos @abitmore