New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactor ticket & user setters #55

Open

soryy708 opened this issue Dec 2, 2018 · 1 comment

Assignees

Labels

Member

soryy708 commented Dec 2, 2018

A lot of ticket setters have the same structure working on slightly different data.
Having so many nearly identical functions is disadvantageous:

Harder to maintain
Worse readability

Refactor it so all of these functions call 1 general function that does the work.

soryy708 added the code quality label

Member Author

soryy708 commented Dec 3, 2018

Some user setters too.

soryy708 changed the title ~~Refactor ticket setters~~ Refactor ticket & user setters

soryy708 added the low priority label

IlanTulepov self-assigned this

lidorz51 self-assigned this

soryy708 removed the low priority label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment