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

Change 101 delegate to status active - Closes #948 #952

Merged
merged 3 commits into from Jul 3, 2018

Conversation

michaeltomasik
Copy link
Contributor

What was the problem?

Delegate 101 didn't have active status

How did I fix it?

Added correct condition

How to test it?

Go to delegate list and check 101 delegate

Review checklist

@michaeltomasik michaeltomasik self-assigned this Jun 25, 2018
@michaeltomasik michaeltomasik added this to Pull Requests in Version 0.10.0 via automation Jun 25, 2018
@michaeltomasik michaeltomasik changed the base branch from development to 0.10.0 June 25, 2018 07:29
@@ -11,7 +11,7 @@ class DelegateStatistics extends React.Component {

let status = '';
if (delegate && delegate.rank) {
status = delegate.rank < 101 ? this.props.t('Active') : this.props.t('Standby');
status = delegate.rank <= 101 ? this.props.t('Active') : this.props.t('Standby');
Copy link
Contributor

Choose a reason for hiding this comment

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

not belonging to the change itself but will be ideal to use constants/voting.maxCountOfVotes

...until testnet and mainnet are aligned
Copy link
Contributor

@slaweet slaweet left a comment

Choose a reason for hiding this comment

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

👍

@slaweet slaweet merged commit 5db7652 into 0.10.0 Jul 3, 2018
Version 0.10.0 automation moved this from Pull Requests to Merged Pull Requests Jul 3, 2018
@slaweet slaweet deleted the 948-rank-101-should-be-active branch July 3, 2018 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Version 0.10.0
  
Merged Pull Requests
Development

Successfully merging this pull request may close these issues.

None yet

3 participants