Skip to content
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.

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

People skills controller cleanup #722

Open
RandomTannenbaum opened this issue May 31, 2024 · 1 comment · May be fixed by #747
Open

People skills controller cleanup #722

RandomTannenbaum opened this issue May 31, 2024 · 1 comment · May be fixed by #747
Labels
plan needs to be planned

Comments

@RandomTannenbaum
Copy link
Collaborator

RandomTannenbaum commented May 31, 2024

Momentan haben wir zwei people-skills Controller. Der controllers/people/people_skills_controller ist für das Updaten von people skills zuständig. Der controllers/people/people_skills_create_controller ist ausschliesslich für das Erstellen von people-skills zuständig. Dadurch haben wir Redundanz.

Die beiden Controller sollten zu einem Controller gemacht und aufgeräumt werden.

Ausserdem wäre es übersichtlicher die route people/12/people_skills auf people/12/skills zu ändern.

@Robin481 Robin481 added the plan needs to be planned label Jun 25, 2024
@RandomTannenbaum RandomTannenbaum linked a pull request Jul 3, 2024 that will close this issue
@RandomTannenbaum
Copy link
Collaborator Author

Stand 03.07.2024
Ich habe damit begonnen den people_skills_controller so umzuschreiben, dass er nun kein Person Controller mehr ist. Denn vorher war er so geschrieben, dass er eigentlich ein person controller ist. Dadurch muss natürlich recht viel Funktionalität in den people_skills views umgeschrieben werden, da die forms dort alle eine person mitgeben und keinen person_skill.

Der nächste Schritt wäre dann, den people_skills_create_controller, der auch wirklich ein people_skills controller ist, mit dem nun umgeschriebenen people_skills_controller zu vereinen, (auf dass sie für immer in glücklicher Zweisamkeit leben) damit wir nur noch einen controller haben.

@ManuelMoeri ManuelMoeri self-assigned this Jul 10, 2024
@ManuelMoeri ManuelMoeri removed their assignment Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plan needs to be planned
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants