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

Added empty cart button in admin cart #3316

Merged
merged 1 commit into from
Aug 30, 2019

Conversation

ericsaupe
Copy link
Contributor

@ericsaupe ericsaupe commented Aug 28, 2019

Description

The frontend has an empty cart button and the API supports an
empty cart call. This adds that functionality to the backend
interface.

Screen Shot 2019-08-28 at 12 06 52 PM

Checklist:

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have updated Guides and README accordingly to this change (if needed)
  • I have added tests to cover this change (if needed)

The frontend has an empty cart button and the API supports an
empty cart call. This adds that functionality to the backend
interface.
Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

Nice work

Copy link
Member

@kennyadsl kennyadsl left a comment

Choose a reason for hiding this comment

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

Thanks @ericsaupe, great work! Going to only change the PR title so it will be easier to add a Changelog entry on the next release.

@kennyadsl kennyadsl changed the title Added empty cart button Added empty cart button in admin cart Aug 29, 2019
@kennyadsl kennyadsl added the changelog:solidus_backend Changes to the solidus_backend gem label Aug 29, 2019
Copy link
Member

@spaghetticode spaghetticode left a comment

Choose a reason for hiding this comment

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

@ericsaupe thank you 👍

@spaghetticode spaghetticode merged commit 44a3e22 into solidusio:master Aug 30, 2019
}

this.model.empty({
success: () => {
Copy link
Contributor

@pelargir pelargir Feb 11, 2020

Choose a reason for hiding this comment

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

Due to the ES6 arrow syntax on this one line, Uglifier has to be put in harmony mode to run Solidus 2.10. This should probably be changed back to ES5 syntax for better compatibility. We shouldn't have to run Uglifier in harmony mode just to get this one line to minify properly.

Copy link
Member

Choose a reason for hiding this comment

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

Good catch, Matt! We would need to either process this file with Babel or update the syntax. I vote for the latter.

@pelargir want to send a PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:solidus_backend Changes to the solidus_backend gem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants