📝 docs(README): update configuration and remove CI section
All checks were successful
build-and-push / docker (push) Successful in 1m8s
All checks were successful
build-and-push / docker (push) Successful in 1m8s
- remove outdated CI/build instructions
- add new environment variable options for configuration
🐛 fix(util): handle notification errors gracefully
- resolve promise instead of rejecting on notification errors
- prevent whole run from failing due to notification issues
This commit is contained in:
parent
0a729d0cbf
commit
9d79f9ac78
2 changed files with 9 additions and 10 deletions
|
|
@ -125,7 +125,8 @@ export const notify = html => new Promise((resolve, reject) => {
|
|||
if (error.message.includes('command not found')) {
|
||||
console.info('Run `pip install apprise`. See https://github.com/vogler/free-games-claimer#notifications');
|
||||
}
|
||||
return reject(error);
|
||||
// don't fail the whole run on notification errors
|
||||
return resolve();
|
||||
}
|
||||
if (stderr) console.error(`stderr: ${stderr}`);
|
||||
if (stdout) console.log(`stdout: ${stdout}`);
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue