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

[SD-1377] Display addresses from countries supported in the current store #11271

Conversation

v10110
Copy link
Contributor

@v10110 v10110 commented Aug 20, 2021

@viezly
Copy link

viezly bot commented Aug 20, 2021

Changes preview:

👀 Review pull request on Viezly

@v10110 v10110 force-pushed the feature/SD-1377-API_Account_Addresses_return_addresses_from_countries_from_current_store branch from ba24aae to 9c3ae0f Compare August 20, 2021 11:10
@v10110 v10110 force-pushed the feature/SD-1377-API_Account_Addresses_return_addresses_from_countries_from_current_store branch from 9c3ae0f to 030a54e Compare August 20, 2021 12:21
context 'when address from countries that are supported in the current store' do
let!(:country) { create(:country, name: 'France') }
let!(:zone) { create(:zone, name: 'EU_VAT', countries: [country], kind: 'country') }
let!(:eu_address) { create(:address, user_id: user.id, country: country) }
Copy link
Member

Choose a reason for hiding this comment

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

@v10110 let's also create another address for this user from a country that isn't supported by this store to test if it doesn't show up

@v10110 v10110 force-pushed the feature/SD-1377-API_Account_Addresses_return_addresses_from_countries_from_current_store branch 2 times, most recently from 33106d0 to e5e2f37 Compare August 24, 2021 07:35
@v10110 v10110 force-pushed the feature/SD-1377-API_Account_Addresses_return_addresses_from_countries_from_current_store branch from e5e2f37 to c62bfd9 Compare August 25, 2021 07:20
@damianlegawiec damianlegawiec merged commit d350b17 into main Sep 3, 2021
@delete-merged-branch delete-merged-branch bot deleted the feature/SD-1377-API_Account_Addresses_return_addresses_from_countries_from_current_store branch September 3, 2021 07:57
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