Skip to content

Commit

Permalink
Added Angular support with LZString as a Factory.
Browse files Browse the repository at this point in the history
  • Loading branch information
gustavomazzoni committed Jul 26, 2016
1 parent 94fc39a commit c58a220
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions libs/lz-string.js
Expand Up @@ -498,4 +498,9 @@ if (typeof define === 'function' && define.amd) {
define(function () { return LZString; });
} else if( typeof module !== 'undefined' && module != null ) {
module.exports = LZString
} else if( typeof angular !== 'undefined' && angular != null ) {
angular.module('LZString', [])
.factory('LZString', function () {
return LZString;
});
}

3 comments on commit c58a220

@gRoussac
Copy link

Choose a reason for hiding this comment

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

Hi Gustavo,

First thanks for your work on this plugin.

Then, do you have any idea why this file is not the same as here ?

https://cdnjs.cloudflare.com/ajax/libs/lz-string/1.4.4/lz-string.js

Is this commit not supposed to be zithin 1.4.4 version ?

thanks

@gRoussac
Copy link

Choose a reason for hiding this comment

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

Here too https://cdn.jsdelivr.net/npm/lz-string@1.4.4/libs/lz-string.js

Did you forget to increment version number zhile qdding this export maybe ?

@gRoussac
Copy link

Choose a reason for hiding this comment

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

Sorry did not see it was a PR.

@pieroxy maybe ? thanks

Please sign in to comment.