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

Angular 6 and rxjs 6 support #166

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Raphcal
Copy link

@Raphcal Raphcal commented Sep 3, 2018

  • Changed version number to 2.0.0 for Semver compatibility
  • Upgraded dependencies version of Angular, RxJS, Zone.js and SystemJS
  • Removed lodash dependency
  • Added unit tests

Fixed "compare" method for null and undefined values

Using const and let when possible

Using triple equals to avoid comparison errors

Fixed unit test by casting "bulb" to allow assignation to SortOrder type

Removed lodash dependency and updated dependencies versions.

Updated systemjs and karma configurations

Fixed code style.

Upgraded version number to 2.0.0

Using SortOrder as argument to better convey the possible values

Added unit tests for new code and fixed onSortChange emissions

Updated NodeJS version for Travis
@Raphcal Raphcal force-pushed the angular6-rxjs6-support branch from 7ad1e51 to a27e6cc Compare September 3, 2018 12:46
@PascalHonegger
Copy link

@Raphcal I know you created this MR over two years ago but I just wanted to thank you for your work. I've integrated your changes into my own fork, I hope you don't mind 😁.

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