search

linnovate / meanio

meanio core module
MIT License
54 stars 73 forks source link

Fix to allow per-asset hash/separator settings #73

Closed jwebbdev closed 8 years ago

jwebbdev commented 9 years ago

Fixes linnovate/mean#1327

Adding logic to parse the url and change the separator makes less sense since the separator is configurable, so just allowing per-asset hash and separator settings will allow it to be overridden in line with the aggregateAsset call. Usage given the issue's case would be:

  myPackage.aggregateAsset('css', 'https://fonts.googleapis.com/css?family=Roboto+Condensed', {
    url: true,
    hash: false,
    weight: 1
  });

or the following would also make the url format valid

  myPackage.aggregateAsset('css', 'https://fonts.googleapis.com/css?family=Open+Sans', {
    url: true,
    separator: '&v=',
    weight: 1
  });