CL Fisher

Results 1061 comments of 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.

@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...