search

puzzle / skills

Open source skill management web application
GNU Affero General Public License v3.0
61 stars 19 forks source link

People skills controller cleanup #722

Open RandomTannenbaum opened 4 months ago

RandomTannenbaum commented 4 months ago

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.

RandomTannenbaum commented 3 months ago

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.