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

Verschieben von Anträgen #6014

Closed
Suweber opened this issue Oct 29, 2019 · 16 comments
Closed

Verschieben von Anträgen #6014

Suweber opened this issue Oct 29, 2019 · 16 comments
Assignees
Labels
PBI Sprint PBI (Product Backlog Item) SPV
Milestone

Comments

@Suweber
Copy link

Suweber commented Oct 29, 2019

Harvest: [0.366 / 21] Projekt - OGIP 56, OGIP 57 und Erweiterungen SPV - Erweiterungen SPV

Bearbeitete Anträge können nicht in ein anderes Dossier/Subdossier verschoben werden.

Stellungnahme 4tw:
Dieses Thema ist im Feedbackforum diskutiert worden. Die Gemeinde Nidau setzt sich stark für die Funktionalität ein, dass Anträge innerhalb des Dossiers verschoben werden können. Köniz hat sich ebenfalls für eine Umsetzung ausgesprochen.

@Suweber Suweber added the SPV label Oct 29, 2019
@njohner
Copy link
Contributor

njohner commented Nov 26, 2019

  • Only allow to move in the same dossier (into subdossiers), because of attached documents and so on
  • Don't forget to update the SQL
  • Also make sure that moving a dossier containing a proposal works correctly

@deiferni
Copy link
Contributor

deiferni commented Dec 2, 2019

currently dossiers with proposals (in submitted state?) in them cannot be moved at all.

@jone jone added ps12 PBI Sprint PBI (Product Backlog Item) Discussion needed Ready for planning It is clear what needs to be done. May or may not be estimated yet. Nachladbar Darf während Sprint nachgeladen werden wenn Arbeitsmangel besteht. and removed Discussion needed labels Jan 7, 2020
@elioschmutz elioschmutz added the Nachgeladen Wurde in den laufenden Sprint geladen (ausserhalb Planning). label Jan 23, 2020
@elioschmutz elioschmutz self-assigned this Jan 23, 2020
@elioschmutz
Copy link
Member

@deiferni Dossiers with proposals can be moved:

screen2

@elioschmutz
Copy link
Member

Es gibt diverse Probleme bei der Umsetzung dieses Issues:

  1. ContentTreeWidget vom Move-Items Formular verwendet die falsche Queries für die Abfrage #6211 Das Contenttreeewidget funktioniert nicht wie gewünscht. Es ist möglich ein Proposal immer in ein anderes Dossier zu verschieben. Der Issue muss erst diskutiert und eine Lösung besprochen werden
  2. Das Contenttreewidget erlaubt es momentan nicht, einen Inhalt in das Root-Dossier zu verschieben. Ich sehe keine einfache Möglichkeit, dies zu ändern. Über das Autocomplete wäre es möglich, aber das funktioniert generell nicht wie es sollte (siehe Punkt 1). Es werden nur Dossiers unterhalb des Root-Dossiers angezeigt. Das verhalten ist Hardcodiert https://github.com/plone/plone.formwidget.contenttree/blob/master/plone/formwidget/contenttree/widget.py#L95.
  3. Dossiers mit eingereichten und abgeschlossenen Anträgen können verschoben werden #6212 Dossiers mit eingereichten Anträgen können verschoben werden (nicht wie von @deiferni vermutet, dass dies eingeschränkt wird)

@lukasgraf lukasgraf added this to the Sprint #12 milestone Jan 27, 2020
@Suweber
Copy link
Author

Suweber commented Jan 28, 2020

@elioschmutz Hab ich das richtig verstanden: Es ist möglich, Anträge in ein anderes Hauptdossier zu verschieben, aber nicht innerhalb des Dossiers ein anderes Subdossier?

@jone jone modified the milestones: Sprint #12, Sprint #13 Jan 28, 2020
@jone jone removed Nachladbar Darf während Sprint nachgeladen werden wenn Arbeitsmangel besteht. Nachgeladen Wurde in den laufenden Sprint geladen (ausserhalb Planning). Discussion needed ps12 Ready for planning It is clear what needs to be done. May or may not be estimated yet. labels Jan 28, 2020
@elioschmutz
Copy link
Member

@Suweber nein, nicht ganz. Momentan ist es möglich, ein Antrag überall hin zu verschieben.

Mit meiner Anpassung wird dies dann nicht mehr möglich sein. Ein Antrag kann danach nur noch innerhalb seines Hauptdossiers verschoben werden können.

@jone
Copy link
Member

jone commented Feb 5, 2020

@Suweber wie besprochen: könntest du das reviewen? Merci!

@Suweber
Copy link
Author

Suweber commented Feb 5, 2020

Issue unvollständig, da Antrag in Dossier via Elemente verschieben "nur" via manuell Eingabe verschoben werden kann. Dossierbaum erscheint nicht.
Da wir nur innerhalb des betroffenen Dossiers verschieben, macht es Sinn das betroffene Dossier bzw. die Subdossiers direkt anzuzeigen.
image

@elioschmutz
Copy link
Member

@Suweber Salut su, in welchem Dossier testest du genau? Hat es subdossiers? Bei mir funktioniert es:

screen1

Aber es gibt ein bekanntes Problem mit dem Widget: Das Root-Dossier kann nicht angewählt werden. Siehe Warnung im PR: #6223 und hier der Issue dazu: #6224

Das Problem ist leider nicht einfach zu lösen und müsste wohl noch im Team besprochen werden. Ich kann nicht beurteilen, ob dies kritisch für den Abschluss ist oder nicht.

@Suweber
Copy link
Author

Suweber commented Feb 6, 2020

@elioschmutz spielt es eine Rolle, welchen Status der Antrag hat? Ich habs hier versucht: https://dev.onegovgever.ch/fd/ordnungssystem/fuehrung/gemeinderecht/uebergeordnete-erlasse/dossier-530#proposals

@elioschmutz
Copy link
Member

@Suweber nein, es spielt keine Rolle.

Genau, dort wo du testest tritt genau der oben beschriebene Fall auf. Es gibt keine Subdossiers. Der Antrag könnte also nur in das Hauptdossier verschoben werden. Das Widget zeigt das Hauptdossier jedoch nicht an. Wenn du ein Subdossier erstellst, sollte dieses angezeigt werden...

@Suweber
Copy link
Author

Suweber commented Feb 6, 2020

stimmt, das Subdossier wird nun angezeigt. Merci! Es ist wohl nicht möglich eine Hilfe einzublenden im Sinne von "keine Subdossiers vorhanden"?

@elioschmutz
Copy link
Member

Von Haus aus gibt es leider nichts. Ich kann es per JS "reinhacken". Eine andere Möglichkeit haben wir momentan nicht... Alternativ übernehmen wir es so wie es ist und werden das Problem dann mit dem Issue #6224 lösen...

Wie seht ihr das? @jone @Suweber

@jone
Copy link
Member

jone commented Feb 6, 2020

@Suweber ich würde es tendenziell mal so sein lassen, weil meine Einschätzung ist dass es relativ Aufwändig ist, einen Mehrwert zu schaffen. Ist das für dich so ok?

@Suweber
Copy link
Author

Suweber commented Feb 7, 2020

ok, ich kann in der Dokumentation ja extra einen Hinweis einbauen, dass im Dossier Subdossier vorhanden sein müssen (um diese anzuzeigen).

@jone
Copy link
Member

jone commented Feb 7, 2020

Ja, wir dokumentieren das. Damit ist dieses PBI abgeschlossen 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PBI Sprint PBI (Product Backlog Item) SPV
Projects
None yet
Development

No branches or pull requests

6 participants