CL Fisher
CL Fisher
@jimmyli97 FYI: We had an urgent fix that required all the senior contributors this week , so we didn't get to look at this again. Hopefully this week will go...
@dorner - does this need a final look-see before merging? (there have been changes since you approved)?
@TheFaizanAnwar FYI: We had an urgent fix that required all the senior contributors this week , so we didn't get to look at this again. Hopefully this week will go...
Hey @dorner -- I think this one is in your court.
@coalest I think we've lost @TheFaizanAnwar -- You've looked at this - do you think you could take it across the line?
@coalest -- Basic functionality is good -- but I'm wondering if we should limit it to 500 characters -- That wasn't specified in the issue, but would be in line...
I agree that's a separate issue. Added it to the inbox to be written up (might be a GFI)
LGTM -- over to @dorner for a final technical look-see.
Difficulty-Intermediate is due to size.
Sounds good !