Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/add env event trigger plip signatures #37

Merged
merged 3 commits into from
Mar 31, 2022

Conversation

dieh
Copy link
Contributor

@dieh dieh commented Mar 29, 2022

No description provided.

@@ -65,4 +65,4 @@ event_triggers:
interval_sec: 10
num_retries: 0
timeout_sec: 60
webhook: http://n8n.enedina.bonde.org/webhook/plip-signatures-mautic
webhook_from_env: WEBHOOK_PLIP_SIGNATURES_MAUTIC_URL
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dieh seria bom adotar o padrão que estamos usando com os outros eventos.

@codecov
Copy link

codecov bot commented Mar 30, 2022

Codecov Report

Merging #37 (33d5f9a) into main (2028e29) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #37   +/-   ##
=======================================
  Coverage   63.28%   63.28%           
=======================================
  Files          41       41           
  Lines         640      640           
  Branches       92       92           
=======================================
  Hits          405      405           
  Misses        210      210           
  Partials       25       25           
Flag Coverage Δ
unittests 63.28% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2028e29...33d5f9a. Read the comment docs.

@dieh dieh force-pushed the feature/add-env-event-trigger-plip-signatures branch from a2c41f8 to 33d5f9a Compare March 30, 2022 18:00
@@ -65,4 +65,4 @@ event_triggers:
interval_sec: 10
num_retries: 0
timeout_sec: 60
webhook_from_env: WEBHOOK_PLIP_SIGNATURES_MAUTIC_URL
webhook: "{{N8N_WEBHOOK_URL}}/plip-signatures-mautic"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lpirola seria esse o formato?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sim @dieh

@lpirola lpirola merged commit 2021b4c into main Mar 31, 2022
@dieh dieh deleted the feature/add-env-event-trigger-plip-signatures branch May 12, 2022 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants