fixed #67, #68 and #69 #70

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
4 participants

This commit patches issues #67, #68 and #69

+ private
+
+ def address_params
+ params.require(:address).permit(:firstname, :lastname, :address1, :address2, :city, :state_name, :zipcode, :country_id, :phone)
Contributor

Senjai commented Mar 25, 2014

Apologies, we need both state_name and state_id to be available.

I added also state_id

Member

iloveitaly commented Jul 13, 2014

Could you rebase these changes to master? Lots of changes have been made for 2.x compatibility and I want to ensure that these are still needed. Thanks!

@coorasse coorasse closed this Jan 18, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment