short icon indicating copy to clipboard operation
short copied to clipboard

Published 20 hours ago • verified publisher icon short-d

[Refactor] Simplify searchShortLink function

Open arberiii opened this issue 4 years ago • 3 comments

What is frustrating you? Function searchShortLink is hard to read, especially the for loop. https://github.com/short-d/short/blob/284d1d6ae7c59549409860bda7c00d3abe9dd703/backend/app/usecase/search/search.go#L68 Your solution Refactor this function appropriately.

arberiii avatar Jun 16 '20 19:06 arberiii

@arberiii Would this be a good first issue? I'd like to work on this, if it hasn't been done already.

hemachandarv avatar Jul 13 '20 12:07 hemachandarv

Yes, you can work on this! Please let me know if you have any trouble with the setup or anything else.

arberiii avatar Jul 13 '20 12:07 arberiii

Cool, I'll start working!

hemachandarv avatar Jul 13 '20 12:07 hemachandarv