update to the connected pharmacies #1140
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Duuude, you're killing me! Just finished making code base version work but this looks like the original csv layout so will change code but I see a problem in that nobody will want to import pharmacies for the entire state (could be hundreds). Recommend add lookup for city and add that as a restriction i.e only import by city. |
|
@sjpadgett , |
|
yes it was
…On Oct 22, 2017 4:49 AM, "Brady Miller" ***@***.***> wrote:
@sjpadgett <https://github.com/sjpadgett> ,
Was this addressed in your recent PR? If so, let me know, and I'll close
this PR.
-brady
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1140 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABxhkXdHHcFoLJlSUdb6c_EccxL8TpcUks5suwGOgaJpZM4QAcze>
.
|
|
thanks! |
|
@bradymiller Sorry but this needs to be merged as it is an updated list plus it match the csv I just did in code. |
|
@sjpadgett , just brought this into codebase. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This list is for importing into the pharmacy list of all the connected Weno pharmacies in the US