#961 Automatically calculate shipping costs based on country selection

Merged
reinhard merged 11 commits from Issue_877_merchanise_wrong_shipping into master 11 months ago
reinhard commented 1 year ago
There is no content yet.
monochromec commented 1 year ago
Collaborator

Let me know if you need support here.

Let me know if you need support here.
reinhard commented 1 year ago
Poster

@monochromec the current state of the pull request is technically functional, but we will use this opportunity to re-evaluate which shipping costs are appropriate for which target countries. I expect this patch to be applied within the upcoming week.

@monochromec the current state of the pull request is technically functional, but we will use this opportunity to re-evaluate which shipping costs are appropriate for which target countries. I expect this patch to be applied within the upcoming week.
max.mehl commented 1 year ago
Owner

At this point, we could also think of using the same stucture as on fsfe.org/promo where we also ask for street, ZIP and city separately. Not sure what makes most sense for the ones who actually send the packages.

At this point, we could also think of using the same stucture as on fsfe.org/promo where we also ask for street, ZIP and city separately. Not sure what makes most sense for the ones who actually send the packages.
vincent requested changes 1 year ago
cgi-bin/weborder.pl
@@ -46,0 +52,4 @@

my $shipping;

given ($country_code) {
vincent commented 1 year ago

I would recommend to avoid given / when because this is an experimental feature. Please see here.

I would recommend to avoid `given` / `when` because this is an experimental feature. Please see [here](https://perldoc.perl.org/perlsyn.html#Experimental-Details-on-given-and-when).
reinhard changed title from WIP: Automatically calculate shipping costs based on country selection to Automatically calculate shipping costs based on country selection 11 months ago
reinhard closed this pull request 11 months ago

Reviewers

vincent requested changes 1 year ago
The pull request has been merged as 46545576be.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.