search

n8n-io / n8n

Free and source-available fair-code licensed workflow automation tool. Easily automate tasks across different services.
https://n8n.io
Other
48.94k stars 7.77k forks source link

Pushover Node Lists twice Timestamp which is a bug, and misses TTL parameter #10984

Open aalmenar opened 1 month ago

aalmenar commented 1 month ago

Bug Description

We you configure the Pushover Node, you can see that timestamp is included twice and TTL is missing which i bealive it was an error when it was implemented.

TTL fields is an integer with the number of seconds to delete the message from the user inbox.

To Reproduce

Add a pushover node, go to Additional Fields, Click on Add Field. There you will have listed twice timestamp which show two timstamp fields.

Expected behavior

To show a single field, and to show on the list of additional fields TTL

Operating System

Debian 12

n8n Version

1.60.1

Node.js Version

20.17.0

Database

SQLite (default)

Execution mode

main (default)

Joffcom commented 1 month ago

Hey @aalmenar,

We have created an internal ticket to look into this which we will be tracking as "GHC-259"

aalmenar commented 1 month ago

You can see the error clearly here: https://github.com/n8n-io/n8n/blob/4584f22a9b16883779d8555cda309fd8bd113f6c/packages/nodes-base/nodes/Pushover/Pushover.node.ts#L239-L260