Skip to content
This repository has been archived by the owner on May 13, 2024. It is now read-only.

Removes total and add optional ac #187

Merged
merged 1 commit into from
Oct 8, 2018
Merged

Removes total and add optional ac #187

merged 1 commit into from
Oct 8, 2018

Conversation

NejcZdovc
Copy link
Contributor

  • removes total line
  • add logic that last element doesn't have border at the bottom

@NejcZdovc NejcZdovc self-assigned this Oct 7, 2018
@NejcZdovc NejcZdovc mentioned this pull request Oct 8, 2018
Merged
11 tasks
jasonrsadler
jasonrsadler approved these changes Oct 8, 2018
View reviewed changes
Copy link
Contributor

@jasonrsadler jasonrsadler left a comment

Choose a reason for hiding this comment

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

lgtm ++

@NejcZdovc NejcZdovc merged commit a04d767 into master Oct 8, 2018
@NejcZdovc NejcZdovc deleted the wallet-summary branch October 8, 2018 16:36
@NejcZdovc NejcZdovc restored the wallet-summary branch October 8, 2018 16:40
@NejcZdovc NejcZdovc deleted the wallet-summary branch October 8, 2018 16:40
@NejcZdovc NejcZdovc restored the wallet-summary branch October 8, 2018 16:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jasonrsadler jasonrsadler jasonrsadler approved these changes

@ryanml ryanml Awaiting requested review from ryanml

Assignees

@NejcZdovc NejcZdovc

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@NejcZdovc @jasonrsadler