-
-
Notifications
You must be signed in to change notification settings - Fork 392
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
OBPIH-5136 enable deletion of a stock request #3630
Conversation
} | ||
} | ||
|
||
def list = { | ||
|
||
def max = params.max ? params.max as int : 10 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to be wrong that it's there. I think this is strictly related to the list redesign changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes that's why I drafted this PR
I believe that during reverting my commit (that I did locally) I did something wrong on the StockMovementController
everything is up to date now
dc817b5
to
65e49da
Compare
<span><i className="fa fa-remove pr-2" /><Translate id="react.default.button.deleteAll.label" defaultMessage="Delete all" /></span> | ||
<span> | ||
<i className="fa fa-remove pr-2" /> | ||
<Translate id="dw" defaultMessage="Cancel Request" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this dw
intentional?
65e49da
to
1f8b8bb
Compare
No description provided.