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

Remove deprecated_address_id column from shipments #4333

Conversation

mamhoff
Copy link
Contributor

@mamhoff mamhoff commented Apr 13, 2022

We have not used this column since we removed all references to it in
2016.

Checklist:

@mamhoff mamhoff force-pushed the remove-deprecated-address-id-column-from-shipments branch from 122197d to d6ba29e Compare April 13, 2022 10:50
We have not used this column since we removed all references to it in
2016.
@mamhoff mamhoff force-pushed the remove-deprecated-address-id-column-from-shipments branch from d6ba29e to e81d213 Compare April 13, 2022 13:52
@waiting-for-dev
Copy link
Contributor

Thanks!

@waiting-for-dev waiting-for-dev merged commit 7f86b91 into solidusio:master Apr 15, 2022
@kennyadsl
Copy link
Member

I'm tempted to revert this change and push it to the next major. Releasing this change with a minor could create more problems than it solves. @mamhoff @waiting-for-dev @tvdeyen what do you think?

@mamhoff
Copy link
Contributor Author

mamhoff commented May 9, 2022

Sure, if you think so.

@waiting-for-dev
Copy link
Contributor

It makes sense.

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.

5 participants