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

fix: result retrieval #44

Merged
merged 2 commits into from
Sep 1, 2020
Merged

fix: result retrieval #44

merged 2 commits into from
Sep 1, 2020

Conversation

AuspeXeu
Copy link
Contributor

Since the plugin is now separated, the result key has changed.

Copy link
Member

@msimerson msimerson left a comment

Choose a reason for hiding this comment

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

Good catch. Would you mind also updating the Changes file and bumping the version in package.json? Then I'll merge and publish.

@AuspeXeu
Copy link
Contributor Author

AuspeXeu commented Sep 1, 2020

Done.

@msimerson msimerson merged commit 4bad030 into haraka:master Sep 1, 2020
msimerson added a commit that referenced this pull request Sep 1, 2020
* rename connect.geoip -> geoip
* fix missing URL in README
* add badges
* code climate improvements
* aggregate results before emitting
* compat with eslint 4 (#13)
* replace node 4 with node 8 testing (#15)
* eslint no-var updates (#17)
* fix lookup exception #18
* add tests for latitude 0 (#22)
* Replace string concatenations with template literals in geoip plugin (#23)
* Fixed issue with missing iso_code
* minor code climate updates
* v1.0.8: add missing 'c' in iso_code. (#29)
* safely access country.iso_code
* replace nodeunit with mocha (#37)
* test: remove conditionals, they mask breaking changes
* es6: use object shorthand
* scripts: use npx to run them
* fix loop (#42)
* add funding
* fix: result retrieval (#44)
* try/catch city & state lookups (#45)
* README: formatting improvements (#46)
* replace travis/appveyor with github actions (#47)
* update badge urls (#48)
* add comment to disabled test
msimerson added a commit that referenced this pull request Sep 3, 2020
msimerson added a commit that referenced this pull request Sep 3, 2020
msimerson added a commit that referenced this pull request Sep 4, 2020
* revert #44, sync flavors, version bump
* cleaner isolation of modules
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