Skip to content

Remove destroy action from TaxCategories Controller#35

Merged
gmacdougall merged 1 commit into
solidusio:masterfrom
seantaylor:remove_tax_categories_controller_decorator
May 12, 2015
Merged

Remove destroy action from TaxCategories Controller#35
gmacdougall merged 1 commit into
solidusio:masterfrom
seantaylor:remove_tax_categories_controller_decorator

Conversation

@seantaylor

Copy link
Copy Markdown

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
Copy Markdown
Contributor

nice

@gmacdougall

Copy link
Copy Markdown
Member

👍

1 similar comment
@athal7

athal7 commented May 8, 2015

Copy link
Copy Markdown

👍

@gmacdougall

Copy link
Copy Markdown
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.

4 participants