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

Add company mapping and api key for ExtremeIpLookup #7104

Merged
merged 5 commits into from Jan 30, 2019

Conversation

npracht
Copy link
Member

@npracht npracht commented Jan 4, 2019

Q A
Bug fix?
New feature? Enhancement
Automated tests included?
Related user documentation PR URL
Related developer documentation PR URL
Issues addressed (#s or URLs)
BC breaks?
Deprecations? None

Description:

When using the Extreme Ip Lookup service, the company name was not mapped with mautic company field. Also, if you subscribed to a Pro service to unlock the limit of API call, the API key was not used.

Steps to test this PR:

A bit annoying because you need a pro account to test the API key. But just reviewing the code should be enough.

  1. Load up this PR
  2. Connect your pro account
  3. Wait some visits on your webpage tracked
  4. See that when the county and city is coming you now also have the company
  5. See that if you exceed 50 requests per minute you have a limitation without API key
  6. Add an API key and see that this limitation is gone.

@npracht npracht added ready-to-test PR's that are ready to test code-review-needed PR's that require a code review before merging enhancement Any improvement to an existing feature or functionality labels Jan 4, 2019
@npracht npracht added this to the 2.15.1 milestone Jan 4, 2019
@npracht npracht requested a review from Woeler January 4, 2019 15:28
@alanhartless alanhartless added this to Needs Testing in 2.15.1 Jan 14, 2019
@Woeler Woeler added this to Ready to Test (first time) in Mautic 2 via automation Jan 18, 2019
Woeler
Woeler previously approved these changes Jan 18, 2019
Copy link
Contributor

@Woeler Woeler left a comment

Choose a reason for hiding this comment

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

Works fine.

@npracht npracht added WIP PR's that are not ready for review and are currently in progress and removed ready-to-test PR's that are ready to test labels Jan 22, 2019
@npracht
Copy link
Member Author

npracht commented Jan 22, 2019

I put it under WIP because the code does not manage the company name to be added on the contact page. Working on it.

Mautic 2 automation moved this from Ready to Test (first time) to Changes Requested / Review Jan 22, 2019
@Enc3phale
Copy link
Contributor

Working now! Key was wrong and PR #7083 solve Company name mapping.

@npracht npracht added pending-test-confirmation PR's that require one test before they can be merged and removed WIP PR's that are not ready for review and are currently in progress labels Jan 22, 2019
Mautic 2 automation moved this from Changes Requested / Review to Ready to Test (first time) Jan 30, 2019
@alanhartless alanhartless merged commit a426db2 into mautic:staging Jan 30, 2019
Mautic 2 automation moved this from Ready to Test (first time) to Merged Jan 30, 2019
@alanhartless alanhartless moved this from Needs Testing to Merged in 2.15.1 Jan 30, 2019
@iamrobreynolds
Copy link

Does this populate Company name field to anonymous contact records?

@npracht
Copy link
Member Author

npracht commented Apr 7, 2019

it does !

@iamrobreynolds
Copy link

@npracht - Awesome and thanks for your response. I have thousands of anonymous contacts with extensive history. I've been using the Maxmind IP service to collect these anonymous contacts. I switched over to ExtremeIpLookup today with no username or password. Should it be collecting the company name and populating the existing anonymous contacts?

@npracht
Copy link
Member Author

npracht commented Apr 8, 2019

@robbiereynolds it is not doing the job for the past contacts. Maybe if they are active again it does work but not sure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code-review-needed PR's that require a code review before merging enhancement Any improvement to an existing feature or functionality pending-test-confirmation PR's that require one test before they can be merged
Projects
No open projects
2.15.1
Merged
Development

Successfully merging this pull request may close these issues.

None yet

5 participants