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

David/1446 delegations filters #1483

Merged
merged 13 commits into from
Oct 26, 2018
Merged

David/1446 delegations filters #1483

merged 13 commits into from
Oct 26, 2018

Conversation

NodeGuy
Copy link
Contributor

@NodeGuy NodeGuy commented Oct 19, 2018

Closes #1446 and closes #1445

Description:

Refactors validator tables by creating new component TableValidators. Has the effect of fixing column sorting in TabMyDelegations. Also removes redundant code from PageStaking.

❤️ Thank you!


  • Added entries in CHANGELOG.md with issue # and GitHub username
  • Reviewed Files changed in the github PR explorer

David Braun added 4 commits October 19, 2018 19:33
…46-delegations-filters

# Conflicts:
#	app/src/renderer/components/staking/PageStaking.vue
#	app/src/renderer/components/staking/TabValidators.vue
#	test/unit/specs/components/staking/PageStaking.spec.js
#	test/unit/specs/components/staking/TabMyDelegations.spec.js
#	test/unit/specs/components/staking/TabValidators.spec.js
#	test/unit/specs/components/staking/__snapshots__/TabValidators.spec.js.snap
@NodeGuy NodeGuy changed the title David/1446 delegations filters WIP: David/1446 delegations filters Oct 19, 2018
@codecov
Copy link

codecov bot commented Oct 19, 2018

Codecov Report

Merging #1483 into develop will increase coverage by 0.03%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #1483      +/-   ##
===========================================
+ Coverage    96.93%   96.96%   +0.03%     
===========================================
  Files           91       92       +1     
  Lines         1727     1712      -15     
  Branches        77       77              
===========================================
- Hits          1674     1660      -14     
  Misses          43       43              
+ Partials        10        9       -1
Impacted Files Coverage Δ
...pp/src/renderer/components/staking/PageStaking.vue 100% <100%> (ø) ⬆️
...c/renderer/components/staking/TabMyDelegations.vue 88.88% <100%> (-11.12%) ⬇️
.../src/renderer/components/staking/TabValidators.vue 100% <100%> (+5%) ⬆️
...rc/renderer/components/staking/TableValidators.vue 100% <100%> (ø)

@NodeGuy NodeGuy changed the title WIP: David/1446 delegations filters David/1446 delegations filters Oct 20, 2018
@NodeGuy
Copy link
Contributor Author

NodeGuy commented Oct 24, 2018

Blocked by #1480.

@NodeGuy NodeGuy added the blocked ✋ issues blocked by other implementations/issues label Oct 24, 2018
jbibla
jbibla previously approved these changes Oct 26, 2018
app/src/renderer/components/staking/TableValidators.vue Outdated Show resolved Hide resolved
@jbibla jbibla self-assigned this Oct 26, 2018
@jbibla jbibla dismissed stale reviews from themself via c423107 October 26, 2018 17:56
@jbibla jbibla merged commit 20cc4c9 into develop Oct 26, 2018
@faboweb faboweb deleted the david/1446-delegations-filters branch October 30, 2018 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked ✋ issues blocked by other implementations/issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

filters on my delegations tab do not work PageStaking and TabValidators have a lot of duplicate code
2 participants