Skip to content

Conversation

eitens-mak
Copy link

Supporting #1385

Needs some more testing and cleanup, but basic functionality is working in the storybook

@eitens-mak eitens-mak marked this pull request as ready for review June 19, 2020 03:44
@eitens-mak
Copy link
Author

One test is failing, but it's also failing in the master branch,

Summary of all failing tests
 FAIL  packages/react-bootstrap-table2/test/store/sort.test.js
  ● Sort Function › sort › should call custom sort function when sortFunc given

    expect(received).toBe(expected)
    
    Expected value to be (using ===):
      6
    Received:
      3
      
      at Object.<anonymous> (packages/react-bootstrap-table2/test/store/sort.test.js:48:34)
          at new Promise (<anonymous>)
      at node_modules/p-map/index.js:46:16
      at processTicksAndRejections (internal/process/task_queues.js:97:5)

@eitens-mak
Copy link
Author

@Chun-MingChen I think you are a maintainer. Anything else to fix so this could be merged?

@AllenFang
Copy link
Member

@eitens-mak I will review it, thanks

@AllenFang
Copy link
Member

BTW, @eitens-mak please change the base to develop branch, master is for release. thanks

@AllenFang AllenFang self-requested a review July 12, 2020 06:44
@@ -71,7 +71,7 @@
"jsdom": "11.2.0",
"jsdom-global": "3.0.2",
"lerna": "2.8.0",
"node-sass": "4.5.3",
"node-sass": "4.12.0",
Copy link
Member

Choose a reason for hiding this comment

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

any reason why upgrade node-sass?

Copy link
Author

Choose a reason for hiding this comment

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

I forget exactly, but I think I couldn't get the dev environment to build on Ubuntu 18.04

@eitens-mak eitens-mak changed the base branch from master to develop July 18, 2020 00:26
@eitens-mak
Copy link
Author

Updated the base branch

@AllenFang
Copy link
Member

thanks!

@eitens-mak
Copy link
Author

eitens-mak commented Aug 22, 2020 via email

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