Skip to content

Collections#1

Open
BabisK wants to merge 14 commits into
masterfrom
collections
Open

Collections#1
BabisK wants to merge 14 commits into
masterfrom
collections

Conversation

@BabisK

@BabisK BabisK commented Jun 17, 2020

Copy link
Copy Markdown
Owner

No description provided.

@BabisK BabisK left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have a look

Comment thread longclaw/checkout/utils.py Outdated
"""
basket_items, current_basket_id = get_basket_items(request)

if not len(basket_items) > 0:

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like you're trying to solve a different problem. If this is the case, open an issue on the upstream and create a fork to solve it. Let's not mesh the issues.

Comment thread longclaw/checkout/api.py Outdated
addresses=address,
shipping_option=shipping_option,
)
try:

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comment on utils.py bellow

@BabisK

BabisK commented Jun 17, 2020

Copy link
Copy Markdown
Owner Author

Also remind me why we need the update from wagtail 2.6 to 2.9

@BabisK BabisK force-pushed the collections branch 2 times, most recently from 2a7aae0 to 8d1e999 Compare August 12, 2021 16:27
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