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

Behavior -> Create -> Error #125

Open
bohdan-vorona opened this issue Feb 8, 2023 · 3 comments
Open

Behavior -> Create -> Error #125

bohdan-vorona opened this issue Feb 8, 2023 · 3 comments
Assignees
Labels
bug Something isn't working status: verified

Comments

@bohdan-vorona
Copy link
Contributor

Describe the bug

behavior-create

To Reproduce

Open the page Behavior -> Create.

@bohdan-vorona bohdan-vorona added the bug Something isn't working label Feb 8, 2023
@bohdan-vorona bohdan-vorona self-assigned this Feb 8, 2023
@bohdan-vorona
Copy link
Contributor Author

@cgsmith I found this bug when I was testing SQL indexes. But I'm not sure about the purpose of this page. The method getList() doesn't exist in the model. But even returning an empty array still is unclear) Needs your review)

create-behaviour

@cgsmith
Copy link
Contributor

cgsmith commented Feb 8, 2023

Yes this is a bug. It shouldn't have made it to master but is only available for admins.

The idea behind behaviors was that a customer could modify an order with the event hooks or with behaviors that we could code in.

I'll have to think a little bit more on what we should do. It'll probably end up being just removing the code.

@bohdan-vorona
Copy link
Contributor Author

@cgsmith Got it, thanks for the explanation)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working status: verified
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

2 participants