Skip to content

Implement per currency base price in table rate shipping#2432

Merged
alecritson merged 11 commits intolunarphp:1.xfrom
ryanmitchell:feature/shipping-tables-base-prices
Apr 9, 2026
Merged

Implement per currency base price in table rate shipping#2432
alecritson merged 11 commits intolunarphp:1.xfrom
ryanmitchell:feature/shipping-tables-base-prices

Conversation

@ryanmitchell
Copy link
Copy Markdown
Contributor

This PR changes the ManageShippingRates component to support a base price for each currency enabled on your store, not just the default price.

Closes #2063

@alecritson
Copy link
Copy Markdown
Collaborator

Thanks @ryanmitchell I wasn't sure if this is ready yet or you was aware of the failing tests?

@ryanmitchell
Copy link
Copy Markdown
Contributor Author

@alecritson yeah working on it, I need to update to target filament 4

@ryanmitchell
Copy link
Copy Markdown
Contributor Author

ryanmitchell commented Apr 7, 2026

@alecritson the current failures in Laravel 11 are coming from kalnoy/nestedset v6.0.8+ which adds whenBooted() even though its not supported in Laravel 11.

I've added a shim to get around it.

@alecritson
Copy link
Copy Markdown
Collaborator

I think the Filament/Resources/ShippingZoneResource/Pages/ManageShippingRates.php table needs updating as currently it will only attempt to format the first price, meaning you get apparently duplicates.

image

@ryanmitchell
Copy link
Copy Markdown
Contributor Author

I've updated it to only show the default currency price

@alecritson alecritson requested a review from glennjacobs April 9, 2026 09:23
@alecritson alecritson merged commit dc1d6a8 into lunarphp:1.x Apr 9, 2026
44 checks passed
@github-project-automation github-project-automation bot moved this from Todo to Done in Roadmap Apr 9, 2026
@ryanmitchell ryanmitchell deleted the feature/shipping-tables-base-prices branch April 9, 2026 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Shipping rate does not allow a price for each currency

4 participants