deps: update vue-good-table

[dated]

Summary

Updates vue-good-table to 2.18.0 and fixes the table components along with it, stopping the wallet from freezing.

This PR also:

• fixes sorting by voted delegate in the wallets table
• fixes sorting in the transactions table

Checklist

• Documentation (if necessary)
• Tests (if necessary)
• Ready to be merged

[ghost]

Thanks for submitting this pull request! A maintainer will review this in the next few days and explicitly select labels so you know what's going on.

If no reviewer appears after a week, a reminder will be sent out.

[dated]

 PASS  __tests__/unit/pages/Profile/ProfileAll.spec.js (9.735s)
  pages > ProfileAll
    √ should render component (58ms)
    aggregatedBalances
      √ should aggregate the sum of the balances of all profiles by network (21ms)
      when there are profiles with the same wallets
        √ should include those wallets only 1 time (21ms)
      when the Ledger has wallets on the current network
        √ should include their balances (24ms)
        when they are included as non-Ledger wallets
          √ should include those wallets only 1 time (18ms)
    totalBalances
      √ should return the sum of balances per network, using their symbols, sorted by quantity descently (17ms)
      when the Ledger has wallets on the current network
        √ should include their balances, sorted by quantity descently (14ms)
    profileBalance
      √ should return the formatted balance of a profile, using the network symbol (18ms)

all tests pass locally 🤔

[dated]

FAIL __tests__/unit/pages/Profile/ProfileAll.spec.js
  ● pages > ProfileAll › totalBalances › should return the sum of balances per network, using their symbols, sorted by quantity descently

    expect(received).toEqual(expected) // deep equality

    - Expected
    + Received

      Array [
    -   "m 500.150909",
    -   "d 0.5201",
    -   "o 0.1219",
    +   "m 500.00",
    +   "d 1.00",
    +   "o 0.00",
      ]

      141 |     it('should return the sum of balances per network, using their symbols, sorted by quantity descently', () => {
      142 |       wrapper = mountPage()
    > 143 |       expect(wrapper.vm.totalBalances).toEqual([
          |                                        ^
      144 |         'm\xa0500.150909',
      145 |         'd\xa00.5201',
      146 |         'o\xa00.1219'

      at Object.toEqual (__tests__/unit/pages/Profile/ProfileAll.spec.js:143:40)

any idea how 225a404 might be causing the rounding issue when run through github actions @faustbrian @alexbarnsley?

[alexbarnsley]

I think there's a problem with pagination 🤔 I can't seem to get past page 2, but the pagination buttons don't seem to change (e.g. it thinks it's still on page 1)

[dated]

Let me check, any specific table or all of em?

[alexbarnsley]

I noticed on transaction and delegate list, so possibly all of them? 🤔

[dated]

Most likely, will fix ASAP..

[dated]

should be fixed now @alexbarnsley

[alexbarnsley]

thanks! I'll be back to checking on PRs in the next few days

[ghost]

A member has approved this PR. A maintainer will merge this PR shortly. If it shouldn't be merged yet, please leave a comment saying so and we'll wait.

Thank you for your contribution!

[ghost]

Your pull request has been merged and marked as tier 4. It will earn you $20 USD.