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

Remove MultiMap from Datamap #5660

Merged
merged 1 commit into from Dec 13, 2018

Conversation

Projects
None yet
2 participants
@swistach
Copy link
Member

swistach commented Dec 12, 2018

Context

MultiMap is no longer necessary. We can replace it with native Map.

How has this been tested?

Handsontable should work as before this change.

Types of changes

  • New feature or improvement (a non-breaking change which adds functionality)

Related issue(s):

  1. #5403

@swistach swistach requested a review from wszymanski Dec 12, 2018

@wojciechczerniak wojciechczerniak referenced this pull request Dec 12, 2018

Closed

Refactor ES5 to ES6 code #5403

11 of 13 tasks complete

@wszymanski wszymanski assigned swistach and unassigned wszymanski Dec 13, 2018

@swistach swistach merged commit 5e841ba into develop Dec 13, 2018

4 of 5 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/codeship Build succeeded
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
security/snyk - package.json (budnix) No new issues
Details
security/snyk - package.json (krzysztofspilka) No new issues
Details

@swistach swistach deleted the feature/multiMapES6 branch Dec 13, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.