Netz39_Vorstand/stammdaten-formular
3
0
Fork 0
Code Issues 2 Pull requests Releases Wiki Activity

ask for a destination if instantiated without a filename #15

Merged
tux merged 3 commits from ask-for-destination into master 2020-07-28 11:14:49 +02:00
Conversation 14 Commits 3 Files changed 1 +16 -7
dkdent commented 2020-03-20 13:17:42 +01:00
Collaborator
Copy link

Nachfrage, wohin die neue Datei gespeichert werden soll, wenn nicht eine bestehende Datei als Input angegeben wurde.

Tab-completion wäre schön, habe ich aber nicht ohne größeren Aufwand hinbekommen, können wir später nachreichen.

Nachfrage, wohin die neue Datei gespeichert werden soll, wenn nicht eine bestehende Datei als Input angegeben wurde. Tab-completion wäre schön, habe ich aber nicht ohne größeren Aufwand hinbekommen, können wir später nachreichen.
tux was assigned by dkdent 2020-03-20 13:17:49 +01:00
tux commented 2020-03-20 17:27:46 +01:00
Owner
Copy link

Es werden trotzdem die Daten von Max Hackerberg geladen (wenn die Datei nicht vorhanden ist, gibt es eine Exception).

Es werden trotzdem die Daten von Max Hackerberg geladen (wenn die Datei nicht vorhanden ist, gibt es eine Exception).
dkdent commented 2020-03-21 10:19:52 +01:00
Author
Collaborator
Copy link

Ja, das war ein Denkfehler... So wie ich das sehe, gibt es drei Fälle:

  1. $ ./edit_data_form.py -> default und nach Pfad fragen
  2. $ ./edit_data_form.py <path/to/existing/file> -> Daten aus File und alten Pfad verwenden
  3. $ ./edit_data_form.py <path/to/non/existing/file> -> was machen wir damit?

Mein Vorschlag ist Error und Ende, was meinst du?

Ja, das war ein Denkfehler... So wie ich das sehe, gibt es drei Fälle: 1. $ ./edit_data_form.py -> default und nach Pfad fragen 2. $ ./edit_data_form.py <path/to/existing/file> -> Daten aus File und alten Pfad verwenden 3. $ ./edit_data_form.py <path/to/non/existing/file> -> was machen wir damit? Mein Vorschlag ist Error und Ende, was meinst du?
tux commented 2020-03-21 18:57:20 +01:00
Owner
Copy link

Ja, das war ein Denkfehler... So wie ich das sehe, gibt es drei Fälle:

  1. $ ./edit_data_form.py -> default und nach Pfad fragen
  2. $ ./edit_data_form.py <path/to/existing/file> -> Daten aus File und alten Pfad verwenden
  3. $ ./edit_data_form.py <path/to/non/existing/file> -> was machen wir damit?

Klingt sinnvoll.

Mein Vorschlag ist Error und Ende, was meinst du?

Ja. Alternativ kann man fragen, ob die Datei erzeugt werden soll. Ich denke aber, dass das mit Variante 1 erstmal ausreichend abgedeckt ist.

> Ja, das war ein Denkfehler... So wie ich das sehe, gibt es drei Fälle: > > 1. $ ./edit_data_form.py -> default und nach Pfad fragen > 2. $ ./edit_data_form.py <path/to/existing/file> -> Daten aus File und alten Pfad verwenden > 3. $ ./edit_data_form.py <path/to/non/existing/file> -> was machen wir damit? Klingt sinnvoll. > Mein Vorschlag ist Error und Ende, was meinst du? Ja. Alternativ kann man fragen, ob die Datei erzeugt werden soll. Ich denke aber, dass das mit Variante 1 erstmal ausreichend abgedeckt ist.
dkdent commented 2020-03-21 18:59:06 +01:00
Author
Collaborator
Copy link

dann baue ich das noch so ein.

dann baue ich das noch so ein.
👍 1
dkdent commented 2020-03-22 21:37:05 +01:00
Author
Collaborator
Copy link

print() scheint nicht zu funktionieren, so lange die curses Umgebung am Start ist, deshalb sieht man meine Statusmeldungen leider gerade nicht.

print() scheint nicht zu funktionieren, so lange die curses Umgebung am Start ist, deshalb sieht man meine Statusmeldungen leider gerade nicht.
tux commented 2020-03-22 22:04:42 +01:00
Owner
Copy link

Das ist richtig - dafür müssen die Dialogoptionen des Toolkits verwendet werden.

Das ist richtig - dafür müssen die Dialogoptionen des Toolkits verwendet werden.
dkdent commented 2020-03-23 09:55:17 +01:00
Author
Collaborator
Copy link

das ist doof... Aktuell tritt der Fehler auf, bevor die GUI generiert ist, da kann ich noch nirgends mit den npyscreen Mitteln hinschreiben. Und wenn ich die Applikation beende, wird auch das ncurses Environment zu gemacht und alles was drin war ist weg.

Ich bin mit den Varianten nicht glücklich.

das ist doof... Aktuell tritt der Fehler auf, bevor die GUI generiert ist, da kann ich noch nirgends mit den npyscreen Mitteln hinschreiben. Und wenn ich die Applikation beende, wird auch das ncurses Environment zu gemacht und alles was drin war ist weg. Ich bin mit den Varianten nicht glücklich.
tux commented 2020-03-23 10:04:04 +01:00
Owner
Copy link

Aktuell tritt der Fehler auf, bevor die GUI generiert ist

Irgendwo zwischen der Initialisierung des Toolkits und der Generierung der GUI müssten wir das ausgeben können.

Oder Du prüfst vor der Initialisierung des Toolkits, ob der Pfad existiert.

> Aktuell tritt der Fehler auf, bevor die GUI generiert ist Irgendwo zwischen der Initialisierung des Toolkits und der Generierung der GUI müssten wir das ausgeben können. Oder Du prüfst vor der Initialisierung des Toolkits, ob der Pfad existiert.
tux closed this pull request 2020-07-23 11:13:59 +02:00
tux reopened this pull request 2020-07-23 11:14:09 +02:00
dkdent commented 2020-07-24 23:16:01 +02:00
Author
Collaborator
Copy link

ich hab zumindest den Konflikt behoben...

ich hab zumindest den Konflikt behoben...
tux commented 2020-07-25 01:22:50 +02:00
Owner
Copy link

Das hatte ich doch auch?

Du hast zwei Branches gemerged - das ist bei einem Feature-Branch problematisch, weil der sich nun nicht mehr verschieben lässt.

Das hatte ich doch auch? Du hast zwei Branches gemerged - das ist bei einem Feature-Branch problematisch, weil der sich nun nicht mehr verschieben lässt.
dkdent commented 2020-07-25 13:36:42 +02:00
Author
Collaborator
Copy link

Mhmh, es kann sein, dass ich noch lokale Änderungen hatte und da etwas durcheinander gekommen ist. Also die letzten paar commits rückgängig machen...

Mhmh, es kann sein, dass ich noch lokale Änderungen hatte und da etwas durcheinander gekommen ist. Also die letzten paar commits rückgängig machen...
dkdent commented 2020-07-27 11:34:03 +02:00
Author
Collaborator
Copy link

Okay, ich kann den source branch nicht aendern... ich mach einen neuen MR auf.

Okay, ich kann den source branch nicht aendern... ich mach einen neuen MR auf.
tux commented 2020-07-28 11:09:30 +02:00
Owner
Copy link

Du hättest hier gar nichts machen müssen - ich hatte den Branch schon aufgeräumt, aber Du das den nicht übernommen.

Du hättest hier gar nichts machen müssen - ich hatte den Branch schon aufgeräumt, aber Du das den nicht übernommen.
tux closed this pull request 2020-07-28 11:14:49 +02:00
tux deleted branch ask-for-destination 2020-07-28 11:14:56 +02:00
tux commented 2020-07-28 11:16:30 +02:00
Owner
Copy link

Du hättest hier gar nichts machen müssen - ich hatte den Branch schon aufgeräumt, aber Du das den nicht übernommen.

Wenn git beim pull Konflikte findet, ist die automatische Konfliktauflösung meistens nicht die richtige Lösung. Insbesondere dann, wenn dabei Merges entstehen.

> Du hättest hier gar nichts machen müssen - ich hatte den Branch schon aufgeräumt, aber Du das den nicht übernommen. Wenn git beim pull Konflikte findet, ist die automatische Konfliktauflösung meistens nicht die richtige Lösung. Insbesondere dann, wenn dabei Merges entstehen.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
tux
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Netz39_Vorstand/stammdaten-formular#15
No description provided.
Delete branch "ask-for-destination"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?