-
Notifications
You must be signed in to change notification settings - Fork 0
10/docu/cleanup unmaintained components #17
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
base: trunk
Are you sure you want to change the base?
Conversation
klees
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hallo @kergomard,
ich finde das soweit erstmal alles grundsätzlich plausibel. Hab allerdings ein paar Anmerkungen:
- ErrorHandlers: Thematisch gehört das mMn fast eher zu Logging als zu Init. Ob wir es da auch unter bekommen ist dann allerdings wohl die andere Frage. Thematisch hat diese
Init-Komponente aus meiner Perspektive eh irgendwie wenig Kohäsion. Im besten Fall ist die beschrieben mit "alles was irgendwie am Anfang passiert". Aber "Password vergessen" gehört mMn eher zuUseroderAuth-Irgendwas, "Public Section" vllt. zuRepository, usw. Das eigentliche "Init" wird nach meinem Verständnis durch die Component Revision langsam obsolet werden. Die Frage ist also, ob wirInitperspektivisch nicht eher entvölkern wollen, statt da neue Sachen reinzuschieben. - YUI könnte sich schon jetzt von alleine erledigen. Wenn sich keiner bewegt funktioniert es auch schon für 10 nicht mehr.
- Bei allen "deprecated" Modulen werden wir danach gefragt werden, ob wir da irgendeinen Plan für haben, wie damit umgegangen werden kann, dass die Komponenten wegfallen. Ich fände es gut, wenn wir was besseres hätten als "Wir lassen es drauf ankommen.". Haben wir da was? Ich denke, dass sollten wir im Meeting bereden.
Ansonsten: Schön, dass wir mal den Service "AssessmentQuestion" im Kern hatten =)
Viele Grüße!
|
Vielen Dank für das Feedback @klees ! Nur eine schnelle Reaktion:
Liebe Grüsse, |
9845f15 to
bd624b1
Compare
Utils: Deprecate ilLegacyFormElementsUtil ILIAS 12 Utils: Deprecate ilStr ILIAS 11 Utils: Deprecate ilUtil ILIAS 12 Utils: Deprecate ilShellUtil ILIAS 12
ee6e952 to
f467fa3
Compare
|
Hallo zusammen Jetzt habe ich mich endlich noch einmal hingesetzt und diesen PR auf den neuesten Stand gebracht. Gemacht ist zusätzlich zum obigen:
Wir haben noch ein paar Komponenten mit einem Unterstrich. Das sind:
Eigentlich würde ich dies gerne so am übernächsten Jour Fixe publizieren, damit wir rechtzeitig dran sind, darum wäre es cool, wenn ihr draufschauen könnet. Ich nehme es auf jeden Fall auf unsere Agenda. Liebe Grüsse, |
Hallo euch allen
Ich habe mich nun mit unseren Komponenten auseinandergesetzt und daraus ist dieser PR geworden.
In aller Kürze die Änderungsvorschläge:
QTIin denTestQuestionPoolverschobenErrorHandlersausExceptionsinInitverschobenCopyWizardinILIASObjectverschobenTypesinBackgroundTasksverschobenChart,Link, Teile vonCSV,HistoryAccordion,Exceptions,PermanentLink,Verification,YUI,jQuery, Rest vonCSV, Teile vonUtilitiesForm, Rest vonUtilitiesIch würde dies als harte Deprecations deklarieren. Zeug fliegt zusammen mit dem Update der Dependencies dann raus.
Grüsse,
Stephan