driesvints / vat-calculator

Handle all the hard stuff related to EU MOSS tax/vat regulations, the way it should be.
MIT License
1.2k stars 88 forks source link

Correct VAT and postal code for Campione d'Italia #138

Closed spaze closed 2 years ago

spaze commented 2 years ago

After checking the changes in #137 I've noticed that Campione d'Italia became part of the EU customs territory on 1 Jan 2020.

That means two relevant changes:

driesvints commented 2 years ago

Thank you @spaze

driesvints commented 10 months ago

@spaze just noticed this PR. This was sent to main but probably should go to 3.x?

spaze commented 10 months ago

@driesvints Possibly, yeah. Can't remember why it's for main though ๐Ÿ˜‡

driesvints commented 10 months ago

@spaze ok, feel free to send it to 3.x if you feel it should go in the current release ๐Ÿ‘

spaze commented 10 months ago

@driesvints Wanted to create a new PR as this one's been merged already but my fork has diverged way too much already and I'm a bit lost in how to file a new PR against your 3.x branch. Do you think you could try merging main into 3.x? I see there are just a few commits besides this one so it shouldn't create many conflicts. If that doesn't work I'll try to think about something ๐Ÿ˜… Thanks!

driesvints commented 10 months ago

@spaze I'll give it a go soon!

spaze commented 10 months ago

Hey @driesvints, I've learned a new trick or two how to sync my fork (had to first switch to the 3.x branch of my fork and then hit the GitHub "Sync fork" button, previously tried that on master which didn't work) and have created #168 by just reapplying the 138.patch to the 3.x branch.

driesvints commented 10 months ago

Cool thanks!