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

Remove destroy action from TaxCategories Controller #35

Conversation

seantaylor
Copy link

It doesn't do anything but the default functionality in
resource controller.

Don't need the specs for #update or #index since they are
using default resource controller actions and should be tested
there.

It doesn't do anything but the default functionality in
resource controller.

Don't need the specs for #update or #index since they are
using default resource controller actions and should be tested
there.
@alexblackie
Copy link
Contributor

nice

@gmacdougall
Copy link
Member

👍

1 similar comment
@athal7
Copy link
Contributor

athal7 commented May 8, 2015

👍

@gmacdougall
Copy link
Member

retest this please

gmacdougall added a commit that referenced this pull request May 12, 2015
…er_decorator

Remove destroy action from TaxCategories Controller
@gmacdougall gmacdougall merged commit f40f849 into solidusio:master May 12, 2015
waiting-for-dev added a commit that referenced this pull request Nov 3, 2022
We're grouping changes by the solidus component they belong to. E.g. (PR
numbers and full changelog range would be linkable):

---
What's Changed

Solidus Core
- Fix bug by @waiting-for-dev in #36
- Add cool feature by @waiting-for-dev in #37

Solidus Admin
- Make something compatible by @waiting-for-dev in #35

Solidus API
- Add endpoint by @waiting-for-dev in #34

Full Changelog: v3.2.0...v3.3.0
---

That configuration makes the release notes follow the same schema that
we're using in the Changelog file [1]. Unfortunately, it's not possible
to change the release or PR templates to make them identical.

[1] - https://github.com/solidusio/solidus/blob/master/CHANGELOG.md
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.

None yet

4 participants