Skip to content

Commit

Permalink
Remove unnecessary Apprise adapter config field (#95)
Browse files Browse the repository at this point in the history
  • Loading branch information
pomeloy authored Mar 27, 2024
1 parent 2123c10 commit 4a9b606
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 9 deletions.
9 changes: 2 additions & 7 deletions lib/notification/adapter/apprise.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import { getJob } from '../../services/storage/jobStorage.js';
import fetch from 'node-fetch';

export const send = ({ serviceName, newListings, notificationConfig, jobKey }) => {
const { priority, server } = notificationConfig.find((adapter) => adapter.id === config.id).fields;
const { server } = notificationConfig.find((adapter) => adapter.id === config.id).fields;
const job = getJob(jobKey);
const jobName = job == null ? jobKey : job.name;
const promises = newListings.map((newListing) => {
Expand All @@ -27,15 +27,10 @@ export const config = {
readme: markdown2Html('lib/notification/adapter/apprise.md'),
description: 'Fredy will send new listings to your Apprise instance.',
fields: {
priority: {
type: 'number',
label: 'Priority',
description: 'The priority of the send notification.',
},
server: {
type: 'text',
label: 'Server',
description: 'The server url to send the notification to.',
description: 'The server URL to send the notification to.',
},
},
};
2 changes: 0 additions & 2 deletions lib/notification/adapter/apprise.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
### Apprise Adapter

Refer to the [instructions](https://github.com/caronc/apprise-api#installation) on how to set up an Apprise instance and how to configure your preferred notification service.

In addition to the Apprise instance, the priority must be defined.

0 comments on commit 4a9b606

Please sign in to comment.