Brad Decker

Results 102 comments of Brad Decker

@abinoda just wanted to add in my 2 cents for the business hour discussion. One thing that has manifested naturally is a rule of law to "not assign reviewers on...

@abinoda do you have any insight into where this will now fall with the GitHub acquisition?

@filipsekulic gonna need a rebase and I assume bump your migration number to avoid the conflict on 073

@danjm you reviewed this previously could you possibly take first pass at this now that it has a migration?

@nikoferro did this pass locally for you? I have a feeling that the "type is too complex" issue will come back here due to how long supported currencies list is.

Looks like junit test results got commited somehow

@benjaminbrgs could you rebase this PR now and we'll get it reviewed. Thank you!

Thanks for your contribution @benjaminbrgs!

@ryanml do you intend on carrying this PR over the finish line? If not can we close it and/or assign this to someone else

@danjm @ryanml this pr is PR - PR1 and needs updated to be considered for inclusion. How should we prioritize this?