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

refs #4678 #1

Merged

Conversation

okazy
Copy link

@okazy okazy commented Oct 12, 2020

@izayoi256 修正ありがとうございます。

テストをしてみたのですが、addSelect がないとソート順が保持されないようで、フロントの商品詳細画面の商品規格がソートされなくなってしまいました。
@orderBy アノテーションでのソートも試みましたが、うまくいかず。
管理画面の機能ですので、エラーが発生することに比べれば商品規格がソートされている重要性は落ちるのではないかと思い、ソートしないよう修正して見ました。

@izayoi256 izayoi256 merged commit 351054b into izayoi256:bugfix/join-product-class Oct 12, 2020
@izayoi256
Copy link
Owner

内容を確認し、マージしました。

@okazy
Copy link
Author

okazy commented Oct 13, 2020

ありがとうございます!

izayoi256 pushed a commit that referenced this pull request Nov 17, 2020
…arch

受注のマルチ検索で、会社名はスペースを除去せず検索
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