Skip to content

Change OrderPaymentPaid event to OrderPaid #142

@diimpp

Description

@diimpp

There is case, when Order may have multiple valid payments with partial amounts, which sums up to Order total. (At least vanilla Sylius supports such behaviour with Order:paymentState->partially_paid and I personally have multi-payment case in one of my projects), so only Order:paymentState should be used to determine, if order is paid, not Payment:state.

Metadata

Metadata

Assignees

No one assigned

    Labels

    EnhancementMinor issues and PRs improving the current solutions (optimizations, typo fixes, etc.).

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions