Skip to content

Mautic logging #1

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

Merged
merged 2 commits into from
Jul 5, 2022
Merged

Mautic logging #1

merged 2 commits into from
Jul 5, 2022

Conversation

roansong
Copy link

@roansong roansong commented Jul 5, 2022

Description

Added logging for calls to mautic_proxy

In the process of moving from Mautic/Acquia to Hubspot.
Starting by adding a log to develop a deeper understanding of requests, in order to prepare for writing the Hubspot part,
which would see requests temporarily be sent to both Acquia and Hubspot before turning the Acquia requests off.

  • Added MauticLog table
  • Added entry for every call to mautic_proxy
Refers/Fixes
Testing

roansong added 2 commits July 5, 2022 08:30
In the process of moving from Mautic/Acquia to Hubspot.
Starting by adding a log to develop a deeper understanding of requests, in order to prepare for writing the Hubspot part,
which would see requests temporarily be sent to both Acquia and Hubspot before turning the Acquia requests off.

* Added MauticLog table
* Added entry for every call to mautic_proxy
@giliomeejg giliomeejg merged commit 2ff0b0f into giliomeejg:master Jul 5, 2022
@@ -0,0 +1,3 @@
from django.shortcuts import render

# Create your views here.
Copy link
Author

Choose a reason for hiding this comment

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

We don't need this file

@@ -0,0 +1,3 @@
from django.test import TestCase

# Create your tests here.
Copy link
Author

Choose a reason for hiding this comment

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

We can test the model with some mock responses, or rather update tests related to the mautic_proxy

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