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
"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
2 participants