Skip to content
This repository has been archived by the owner on Jan 14, 2022. It is now read-only.

Commit

Permalink
Merge pull request #71 from cbarkerms/master
Browse files Browse the repository at this point in the history
remove hard requirement for w3c manifest, this finally addresses the issue #35
  • Loading branch information
boyofgreen committed Apr 22, 2016
2 parents 2c42e4f + a4f3203 commit 782a3de
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions scripts/updateConfigurationBeforePrepare.js
Original file line number Diff line number Diff line change
Expand Up @@ -587,11 +587,12 @@ module.exports = function (context) {
var manifestPath = path.join(projectRoot, 'manifest.json');
fs.readFile(manifestPath, function (err, data) {
if (err) {
logger.error('Failed to read manifest at \'' + manifestPath + '\'.');
return task.reject(err);
logger.warn('Failed to read manifest at \'' + manifestPath + '\'. Proceeding to point config.xml to sample url of https://www.npmjs.com/package/cordova-plugin-hostedwebapp.');
data = JSON.stringify({ 'start_url': 'https://www.npmjs.com/package/cordova-plugin-hostedwebapp', 'short_name' : 'PlaceholderSite'});
}

var manifestJson = data.toString().replace(/^\uFEFF/, '');

var appManifestPath = path.join(projectRoot, 'www', 'manifest.json');
fs.writeFile(appManifestPath, manifestJson, function (err) {
if (err) {
Expand Down

0 comments on commit 782a3de

Please sign in to comment.