unleash icon indicating copy to clipboard operation
unleash copied to clipboard

fix: Technical debt

Open Tymek opened this issue 2 years ago • 5 comments

Things that we would like to improve in the codebase.

  • [x] #2715
  • [ ] React Table update (v8) @Tymek
  • [ ] ConditionallyRender @Tymek
  • [x] Integration testing @kwasniew (https://github.com/Unleash/unleash/pull/2422)
  • [x] Enable eslint for frontend (again) (https://github.com/Unleash/unleash/pull/2562)
  • [x] https://github.com/Unleash/unleash/issues/2306 @chriswk
  • [x] #2606

Anything else?

Please send me things to add or comment below. Anything that you would like to prioritize updating/fixing/enhancing. Non-breaking changes here, breaking changes for v5 in https://github.com/Unleash/unleash/issues/1265

Tymek avatar Dec 05 '22 09:12 Tymek

Can any of the above be picked up by a beginner to the repo @Tymek ? I can give one of these a shot if needed but I see PRs against a lot of the items already :)

ShaswatPrabhat avatar Dec 06 '22 02:12 ShaswatPrabhat

@ShaswatPrabhat https://github.com/Unleash/unleash/issues/2606 is a good first issue. This doesn't need to be done in full to be helpful. Every refactored file helps. Most of everything else is medium to advanced.

Tymek avatar Dec 06 '22 12:12 Tymek

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Feb 18 '23 20:02 stale[bot]

is ConditionallyRender finished @Tymek?

gastonfournier avatar Apr 19 '23 11:04 gastonfournier

@gastonfournier No. I have a script to do it, I need couple of hours to parse a codebase, fix edge cases.

Tymek avatar Apr 19 '23 11:04 Tymek