- Issue created by @pfrenssen
- 🇦🇺Australia darvanen Sydney, Australia
Absolutely. Do you think it can be done in the same code or do we need to start a new major to support it?
- Assigned to pfrenssen
- 🇧🇬Bulgaria pfrenssen Sofia
I am going to try it out today. I reviewed the diff between Address 1.12 and 2.0.0 and there differences are minimal. The API remains unchanged apart from some very minor differences which will probably not affect us. The data model has had an addition of a new column "address_line3".
A new major version is probably not necessary.
- Status changed to Needs review
5 months ago 7:54pm 24 July 2024 - 🇧🇬Bulgaria pfrenssen Sofia
Simply adding the new field works fine. I've tested it with both Address 1.12.0 and 2.0.2. When using Address v1, the field will simply always return
NULL
. - Status changed to RTBC
5 months ago 1:41am 26 July 2024 - 🇦🇺Australia darvanen Sydney, Australia
Nice one! Thank you very much.
Based on you covering off the only issue I can think of ocurring I'm going to commit this, the code is good. Would love a test but I think that can be a follow-up.
-
darvanen →
committed 0140980b on 2.x authored by
pfrenssen →
Issue #3462446 by pfrenssen, darvanen: Support Address 2.x
-
darvanen →
committed 0140980b on 2.x authored by
pfrenssen →
- Status changed to Fixed
5 months ago 1:41am 26 July 2024 Automatically closed - issue fixed for 2 weeks with no activity.