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

Issues 2164 #2219

Merged
merged 11 commits into from
Oct 17, 2017
Merged

Issues 2164 #2219

merged 11 commits into from
Oct 17, 2017

Conversation

raftaar1191
Copy link
Contributor

@raftaar1191 raftaar1191 commented Oct 17, 2017

Description

PR for issues #2164

How Has This Been Tested?

Tested by uploading the CSV which has postal code as column and it is auto selected as ZIP in the CSV mapping

Screenshots (jpeg or gifs if applicable):

image

image

Types of changes

New feature (non-breaking change which adds functionality)

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows has proper inline documentation.

@raftaar1191
Copy link
Contributor Author

@ravinderk @DevinWalker

Please review the PR and merge

@ravinderk
Copy link
Collaborator

@raftaar1191 Good Job 👍

@ravinderk ravinderk merged commit e8765e1 into impress-org:release/1.8.15 Oct 17, 2017
@raftaar1191
Copy link
Contributor Author

Thanks @ravinderk and thanks for review and updating my PR too

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.

None yet

2 participants