Validate address#1768
Validate address#1768angaither merged 6 commits intoDoSomethingArchive:devfrom angaither:validate-address
Conversation
angaither
commented
Apr 14, 2014
angaither
commented
- Add system settings form
- Valid XML according the UPS guidelines
- Makes post request to UPS api & print the results.
- create menu callback for an ajax request - format XML request Refs #74
There was a problem hiding this comment.
Why not store this variable with the same prefix, dosomething_ups .. ? Or additionally use the dosomething_user module prefix as well to better specify where its being set?
|
@aaronschachter at least I'm consistent. 😄 |
There was a problem hiding this comment.
Is all of this new code for the USPS or UPS? If USPS, please update the variable and permission names accordingly (or fix this comment to standardize)
There was a problem hiding this comment.
The fake xml stuff was originally for UPS, will be updated in a new PR.
Formats all XML as UPS defines. Return the results from the api call to the page.
There was a problem hiding this comment.
Same question, all this code is for USPS, not UPS right? Can we do a global replace for anything UPS as USPS if so?
There was a problem hiding this comment.
It's all for UPS, don't think there's anything referencing USPS anymore.
2 participants
