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

Kursbeschreibung vom Adminbereich beim Kurs abfüllen #100

Closed
RolandStuder opened this Issue Oct 18, 2016 · 10 comments

Comments

Projects
None yet
3 participants
@RolandStuder
Member

RolandStuder commented Oct 18, 2016

IST: Es kann im Adminbereich zu einem Kurstyp eine Kursbeschreibung erfasst werden (Text unter Allgemein Informationen). Allerdings wird diese nirgendwo angezeigt.

SOLL:

  • Die Kursbeschreibung aus dem Adminbereich soll wird unbenannt auf "Standardbeschreibung"
  • Sobald ich ich auf dem Formular Kurs editieren oder Kurs erstellen den Kurstyp wähle, wird das bestehende Beschreibungsfeld automatisch ausgefüllt, sofern es leer ist.
  • Wenn es nicht leer ist und ich den Kurstyp wähle, dann erscheint bei der Beschreibung ein Link "Standardbeschreibung einsetzen", der Text im Feld Beschreibung wird Standardtext ergänzt (hinter den bestehenden Text).
  • Sicherstellen, dass die Beschreibung damit jetzt nicht irgendwo dupliziert wird (z. B. bei PBS).

edit: https://wearecube.mybalsamiq.com/projects/hitobito/edit/100%20Standardkursbeschreibung


Originaltext vom Issue:

Mein Favorit:

  • Sobald ich Kurstyp wähle, automatisch Beschreibungsfeld ausfüllen, wenn es leer ist.
  • Wenn es nicht leer ist und ich den Kurstyp wähle, dann erscheint bei der Beschreibung ein Link "Standardtext einsetzten", der Text im Feld Beschreibung wird dann vom Standardtext überschrieben.

Günstigste Variante:

  • Einfach beim Kurs in einem zusätzlichen Bereich den "Standardtext" anzeigen, allerdings lässt sicher dieser dann nicht ändern.

Originalissue:
http://feedback.jubla.ch/issues/60

@fossil12

This comment has been minimized.

Show comment
Hide comment
@fossil12

fossil12 Oct 24, 2016

@RolandStuder Wir finden deinen Favorit super!

fossil12 commented Oct 24, 2016

@RolandStuder Wir finden deinen Favorit super!

@codez

This comment has been minimized.

Show comment
Hide comment
@codez

codez Oct 27, 2016

Contributor

Die Allgemeinen Informationen zur Kursart haben doch nur bedingt mit der Beschreibung des Kurses zu tun? Bei der PBS werden diese Informationen, falls vorhanden, direkt unter der Kursart dargestellt. Ich würde auch hier für eine einheitliche Lösung plädieren.

Contributor

codez commented Oct 27, 2016

Die Allgemeinen Informationen zur Kursart haben doch nur bedingt mit der Beschreibung des Kurses zu tun? Bei der PBS werden diese Informationen, falls vorhanden, direkt unter der Kursart dargestellt. Ich würde auch hier für eine einheitliche Lösung plädieren.

@fossil12

This comment has been minimized.

Show comment
Hide comment
@fossil12

fossil12 Oct 28, 2016

Ich bin nicht direkt in den entsprechenden Gremien dabei, darum weiss ich nur bedingt wie es funktioniert, aber so wie ich das verstanden habe werden die Beschreibungen von der Bulei and die Ast geschickt und dann dort 1:1 für jeden Kurs von Hand eingetragen. Dies sollte mit dieser Lösung verhindert werden. Die Beschreibungen sind je nach Kursart verschieden (wenn ich mich da nicht irre).

fossil12 commented Oct 28, 2016

Ich bin nicht direkt in den entsprechenden Gremien dabei, darum weiss ich nur bedingt wie es funktioniert, aber so wie ich das verstanden habe werden die Beschreibungen von der Bulei and die Ast geschickt und dann dort 1:1 für jeden Kurs von Hand eingetragen. Dies sollte mit dieser Lösung verhindert werden. Die Beschreibungen sind je nach Kursart verschieden (wenn ich mich da nicht irre).

@RolandStuder

This comment has been minimized.

Show comment
Hide comment
@RolandStuder

RolandStuder Oct 30, 2016

Member

@codez Das habe ich mir angeschaut, bei der PBS wird das gar nicht verwendet. Das Feld ist auch nicht nachvollziehbar, was damit passiert, deshalb migrieren wir dies zur neuen Funktion. Das Feld heute hat eigentlich gar keine wirklich Zweck 😆

Member

RolandStuder commented Oct 30, 2016

@codez Das habe ich mir angeschaut, bei der PBS wird das gar nicht verwendet. Das Feld ist auch nicht nachvollziehbar, was damit passiert, deshalb migrieren wir dies zur neuen Funktion. Das Feld heute hat eigentlich gar keine wirklich Zweck 😆

@RolandStuder RolandStuder self-assigned this Oct 30, 2016

@RolandStuder RolandStuder changed the title from Kursbeschreibung im Adminbereich erfassen und anzeigen to Kursbeschreibung vom Adminbereich beim Kurs abfüllen Nov 2, 2016

@RolandStuder

This comment has been minimized.

Show comment
Hide comment
@RolandStuder

RolandStuder Nov 10, 2016

Member

Das war bereits Diskussion in #27

Member

RolandStuder commented Nov 10, 2016

Das war bereits Diskussion in #27

dinuu added a commit to Zeilenwerk/hitobito that referenced this issue Nov 18, 2016

dinuu added a commit to Zeilenwerk/hitobito that referenced this issue Nov 18, 2016

dinuu added a commit to Zeilenwerk/hitobito that referenced this issue Nov 18, 2016

dinuu added a commit to Zeilenwerk/hitobito that referenced this issue Nov 18, 2016

@RolandStuder

This comment has been minimized.

Show comment
Hide comment
@RolandStuder
Member

RolandStuder commented Nov 21, 2016

@codez

This comment has been minimized.

Show comment
Hide comment
@codez

codez Dec 8, 2016

Contributor

@RolandStuder Soll das nur in den Jubla Wagon und nicht in den Core?

Contributor

codez commented Dec 8, 2016

@RolandStuder Soll das nur in den Jubla Wagon und nicht in den Core?

@RolandStuder

This comment has been minimized.

Show comment
Hide comment
@RolandStuder

RolandStuder Dec 13, 2016

Member

@dinuu Das sollte in den Core.

Member

RolandStuder commented Dec 13, 2016

@dinuu Das sollte in den Core.

@codez

This comment has been minimized.

Show comment
Hide comment
@codez

codez Dec 21, 2016

Contributor

Merged, wird nächstens auf die Integrationsumgebung deployt (ab Version 1.14).

Contributor

codez commented Dec 21, 2016

Merged, wird nächstens auf die Integrationsumgebung deployt (ab Version 1.14).

@RolandStuder

This comment has been minimized.

Show comment
Hide comment
@RolandStuder

RolandStuder Mar 10, 2017

Member

Fertig für Review.

Meine Beobachtungen:

  • Link heisst nur "Standardbeschreibung" und ist damit nicht selbst erklärend -> sollte sein "Standardbeschreibung einsetzen"
  • Link erscheint auch, wenn gar keine Standardbeschreibung für den Kurstyp vorhanden ist. Besser wäre, wenn nur bei vorhandener Standardbeschreibung der Link erscheinen würde.
Member

RolandStuder commented Mar 10, 2017

Fertig für Review.

Meine Beobachtungen:

  • Link heisst nur "Standardbeschreibung" und ist damit nicht selbst erklärend -> sollte sein "Standardbeschreibung einsetzen"
  • Link erscheint auch, wenn gar keine Standardbeschreibung für den Kurstyp vorhanden ist. Besser wäre, wenn nur bei vorhandener Standardbeschreibung der Link erscheinen würde.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment