Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changes to CartCondition helper #16

Closed
wants to merge 3 commits into from

Conversation

ryanmitchell
Copy link
Member

Flame related changes: tastyigniter/TastyIgniter#481

@sampoyigi
Copy link
Member

I'm guessing this is no longer needed, can you close this and related PRs

@ryanmitchell
Copy link
Member Author

This relates to the coupon changes - it's still needed to allow the calculation of the discount based on what items are eligible (due to category or product restrictions)

@sampoyigi
Copy link
Member

Ok, let me see if i can come up with a different approach, also have you transferred the extension over to TI? I want to start the review process on time

@ryanmitchell
Copy link
Member Author

Havent transferred yet, was waiting for you to come back on the questions from the core models - I'll work on the events approach over the weekend or at the start of next week.

@ryanmitchell
Copy link
Member Author

@sampoyigi what about this sort of approach? In summary:

add limitations array to CartCondition
which we then loop over in processValue() to recalculate total on the basis of
see here for the coupon side of it:

tastyigniter/ti-ext-coupons@e91cd6c

@sampoyigi
Copy link
Member

Got a different approach in mind, I will get started on it and push once I have it pieced together.

@ryanmitchell
Copy link
Member Author

👍

@sampoyigi
Copy link
Member

See it here #19, some improvements can definitely be made.

@ryanmitchell
Copy link
Member Author

Am I good to close this (nothing is depending on it?)

@sampoyigi
Copy link
Member

Yup!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants