Commit graph

6 commits

Author SHA1 Message Date
Iván Ovejero 70ae90fa3c
refactor: Apply more eslint-plugin-n8n-nodes-base autofixable rules ()
*  Update `lintfix` script

* 👕 Remove unneeded lint exceptions

* 👕 Run baseline `lintfix`

* 👕 Apply `node-param-description-miscased-url` ()

* 👕 Apply `rule node-param-placeholder-miscased-id` ()

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

* 👕 Apply `node-param-option-name-wrong-for-upsert` ()

* 👕 Apply `node-param-min-value-wrong-for-limit` ()

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

* Apply `node-param-display-name-wrong-for-dynamic-options` ()

* 🔨 fix

*  Fix `Assigned To` fields

Co-authored-by: Michael Kret <michael.k@radency.com>

* 👕 Apply `rule node-param-default-wrong-for-number` ()

* 👕 Apply `node-param-default-wrong-for-string` ()

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

* Apply `node-param-display-name-miscased` ()

* 🔨 fix

* 🔨 exceptions

*  review fixes

* 👕 Apply `node-param-description-lowercase-first-char` ()

*  fix

*  review fixes

*  fix

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

* 👕 Apply `node-param-description-wrong-for-dynamic-options` ()

* Rule working as intended

* Add rule

* 🔥 Remove repetitions

* 👕 Add exceptions

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

* 👕 Small fix for `node-param-description-wrong-for-dynamic-options`

* 👕 Apply `node-param-default-wrong-for-fixed-collection` ()

* 👕 Apply `node-param-description-line-break-html-tag` ()

* 👕 Run baseline `lintfix`

* 👕 Apply `node-param-options-type-unsorted-items` ()

*  fix

* 🔨 exceptions

* Add exception for Salesmate and Zoom

Co-authored-by: Michael Kret <michael.k@radency.com>
Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

*  Restore `lintfix` command

Co-authored-by: Omar Ajoue <krynble@gmail.com>
Co-authored-by: Michael Kret <88898367+michael-radency@users.noreply.github.com>
Co-authored-by: agobrech <45268029+agobrech@users.noreply.github.com>
Co-authored-by: Michael Kret <michael.k@radency.com>
Co-authored-by: brianinoa <54530642+brianinoa@users.noreply.github.com>
2022-06-03 19:23:49 +02:00
Iván Ovejero 6dcdb30bf4
refactor: Apply more nodelinting rules ()
* ✏️ Alphabetize lint rules

* 🔥 Remove duplicates

*  Update `lintfix` script

* 👕 Apply `node-param-operation-without-no-data-expression` ()

* 👕 Apply `node-param-operation-without-no-data-expression`

* 👕 Add exceptions

* 👕 Apply `node-param-description-weak` ()

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

* 👕 Apply `node-param-option-value-duplicate` ()

* 👕 Apply `node-param-description-miscased-json` ()

* 👕 Apply `node-param-display-name-excess-inner-whitespace` ()

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

* 👕 Apply `node-param-type-options-missing-from-limit` ()

* Rule workig as intended

* ✏️ Uncomment rules

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

* 👕 Apply `node-param-option-name-duplicate` ()

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

* 👕 Apply `node-param-description-wrong-for-simplify` ()

*  fix

*  exceptions

*  changed rule ignoring from file to line

* 👕 Apply `node-param-resource-without-no-data-expression` ()

* 👕 Apply `node-param-display-name-untrimmed` ()

* 👕 Apply `node-param-display-name-miscased-id` ()

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

* 👕 Apply `node-param-resource-with-plural-option` ()

* 👕 Apply `node-param-description-wrong-for-upsert` ()

*  fix

*  replaced record with contact in description

*  fix

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

* 👕 Apply `node-param-option-description-identical-to-name` ()

* 👕 Apply `node-param-option-name-containing-star` ()

* 👕 Apply `node-param-display-name-wrong-for-update-fields` ()

* 👕 Apply `node-param-option-name-wrong-for-get-all` ()

*  fix

*  exceptions

* 👕 Apply node-param-display-name-wrong-for-simplify ()

* Rule working as intended

* Uncomented other rules

* 👕 Undo and add exceptions

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

*  Alphabetize lint rules

*  Restore `lintfix` script

Co-authored-by: Michael Kret <88898367+michael-radency@users.noreply.github.com>
Co-authored-by: agobrech <45268029+agobrech@users.noreply.github.com>
2022-05-20 23:47:24 +02:00
Iván Ovejero 63b6c9f128
refactor: Apply more eslint-plugin-n8n-nodes-base autofixable rules ()
* ✏️ Alphabetize rules

* 🔖 Update version

*  Update lintfix command

*  Run baseline lintfix

* 📦 Update package-lock.json

* 👕 Apply `node-param-description-untrimmed` ()

* Removing unneeded backticks ()

* 👕 Apply node-param-description-wrong-for-return-all ()

* 👕 Apply node-param-description-missing-limit ()

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

* 👕 Apply node-param-description-excess-final-period ()

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

* 👕 Apply node-param-description-unencoded-angle-brackets ()

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

* 👕 Apply node-param-description-url-missing-protocol ()

* 👕 Apply `node-param-description-miscased-id` ()

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

* 👕 Apply node-param-description-wrong-for-limit ()

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

* 👕 Apply node-param-description-wrong-for-ignore-ssl-issues ()

* 👕 Apply rule

*  Restore lintfix script

*  Restore lintfix script

Co-authored-by: agobrech <45268029+agobrech@users.noreply.github.com>
Co-authored-by: Omar Ajoue <krynble@gmail.com>
Co-authored-by: Michael Kret <88898367+michael-radency@users.noreply.github.com>
2022-05-06 23:01:25 +02:00
Iván Ovejero 0448feec56
refactor: Apply eslint-plugin-n8n-nodes-base autofixable rules ()
*  Initial setup

* 👕 Update `.eslintignore`

* 👕 Autofix node-param-default-missing ()

* 🔥 Remove duplicate key

* 👕 Add exceptions

* 📦 Update package-lock.json

* 👕 Apply `node-class-description-inputs-wrong-trigger-node` ()

* 👕 Apply `node-class-description-inputs-wrong-regular-node` ()

* 👕 Apply `node-class-description-outputs-wrong` ()

* 👕 Apply `node-execute-block-double-assertion-for-items` ()

* 👕 Apply `node-param-default-wrong-for-collection` ()

* 👕 Apply node-param-default-wrong-for-boolean ()

* Autofixed default missing

* Autofixed booleans, worked well

*  Fix params

*  Undo exempted autofixes

* 📦 Update package-lock.json

* 👕 Apply node-class-description-missing-subtitle ()

*  Fix missing comma

* 👕 Apply `node-param-default-wrong-for-fixed-collection` ()

* 👕 Add exception for `node-class-description-missing-subtitle`

* 👕 Apply `node-param-default-wrong-for-multi-options` ()

* 👕 Apply `node-param-collection-type-unsorted-items` ()

* Missing coma

* 👕 Apply `node-param-default-wrong-for-simplify` ()

* 👕 Apply `node-param-description-comma-separated-hyphen` ()

* 👕 Apply `node-param-description-empty-string` ()

* 👕 Apply `node-param-description-excess-inner-whitespace` ()

* Rule looks good

* Add whitespace rule in eslint config

* :zao: fix

* 👕 Apply `node-param-description-identical-to-display-name` ()

* 👕 Apply `node-param-description-missing-for-ignore-ssl-issues` ()

*  Revert ":zao: fix"

This reverts commit ef8a76f3df.

* 👕 Apply `node-param-description-missing-for-simplify`  ()

* 👕 Apply `node-param-description-missing-final-period` ()

* Rule working as intended

* Add rule to eslint

* 👕 Apply node-param-description-missing-for-return-all ()

*  Restore `lintfix` command

Co-authored-by: agobrech <45268029+agobrech@users.noreply.github.com>
Co-authored-by: Omar Ajoue <krynble@gmail.com>
Co-authored-by: agobrech <ael.gobrecht@gmail.com>
Co-authored-by: Michael Kret <michael.k@radency.com>
2022-04-22 18:29:51 +02:00
Iván Ovejero a783288338
Enforce type checking in all node params () 2021-12-03 09:44:16 +01:00
Jan 4573d503dc
Add matrix integration ()
* Added Matrix integration node

* fix: Improve code quality and add new operation

- Changed operation names to match casing (all camelCase)
- Ordering operation names to be alphabetical
- Creating a read all operation to fetch all messages from a room
- Added node subtitle

* fix: add element index so that expressions work on multiple items

* feature: added possibility to upload and send media to Matrix

- Also replacing Promises.all() + Array.map() For a regular for as it messes up ordering

* refactor: merging upload + send Media in a single action

* refactor: improved code quality and endpoints

- Removed sync entirely as a better way to retrieve messages is now
implemeented
- Added rooms dropdown to operations
- Added option to paginate or retrieve all room messages
- Removed option to upload media from text contents. Only files are
accepted now
- Room members has bem moved from the Rooms resource to a standalone
with Get All operation

*  Small improvements

*  Added filters to get messages

*  Minor improvements to Matrix-Integration

Co-authored-by: Omar Ajoue <krynble@gmail.com>
Co-authored-by: ricardo <ricardoespinoza105@gmail.com>
2020-10-12 10:05:16 +02:00