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

AccessMana is now Mana1 #2271

Merged
merged 5 commits into from
Jun 20, 2022
Merged

AccessMana is now Mana1 #2271

merged 5 commits into from
Jun 20, 2022

Conversation

karimodm
Copy link
Contributor

Description of change

Rebased #2267 on top of develop

karimodm added 4 commits June 13, 2022 14:49
* package/mana/access* all gone
* consensusbase renamed to manabase implementing base interface for both access and consensus mana
* consensusbasevector renamed to manabasevector implementing basevector interface for both access and consensus mana
* both mana vectors are now maintained using 2 instances of ManaBaseVector (former ConsensusBaseVector)
* update methods removed from both interfaces and implementations
* SetCoefficients gone, along with corresponding mana plugin parameters
    *
* EffectiveValue methods removed, only BaseValue remains
* manarefresher is gone
    * DelegationAddress stuff managed by the plugin also gone
        * Reclaim functionality removed
    * corresponding WebAPI endpoints and response fields removed
    * cli-wallet delegation & reclaim removed
    * occurrences removed from Ansible and docker-compose files
* GetMana and GetManaMap methods do not receive and optional time anymore, but they still return a timestamp which is always set to time.Now(). This timestamp should be tied to the epoch timestamp for the committed mana vector.

 Conflicts:
	packages/mana/accessbasevector.go
	packages/mana/manabasevector.go
	packages/mana/parameters.go
	plugins/core.go
@karimodm karimodm requested a review from piotrm50 June 13, 2022 12:55
Copy link
Contributor

@piotrm50 piotrm50 left a comment

Choose a reason for hiding this comment

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

lgtm

@karimodm karimodm merged commit 57ba081 into develop Jun 20, 2022
@karimodm karimodm deleted the feat/access-mana1 branch June 20, 2022 08:41
This was referenced Jun 22, 2022
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