CL Fisher
CL Fisher
@neenu-chacko Ok .. we'll mark it as open and point to your PR. Please do check back when you can -- sometimes this sort of 'last mile' work isn't picked...
Note: this is a PR on an issue that already had a PR on it.
@princekumarg12 There was already a PR in progress for this one, so I think we'll go with that one, if it works.
Hey @bdeanhardt -- Thank you! -- we'll take a look. One thing for future -- we usually put a PR for each issue we work on -- it's easier for...
Also for future -- it's a better practice to make a branch, rather than making the changes in your main.
@bdeanhardt Could you make the order "human alphabetical" -- i.e. case insensitive, aka lower case alphabetical?, rather than "computer alphabetical", which orders all the lowercase after all the upper case,...
@bdeanhardt Functionally, this looks great! -- just waiting for those automated tests, then I'll pass it over to our tech lead for a final technical review.
Will do. Thanks!
You may have it!
@manuel1280 Looks pretty good to me -- I might move the total items down a line, because the prices form a natural group, but I wouldn't reject the PR if...