-
Notifications
You must be signed in to change notification settings - Fork 0
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
#59 Alles Material löschen Funktion hinzugefügt #67
#59 Alles Material löschen Funktion hinzugefügt #67
Conversation
Button um alles Material der Abteilung zu löschen auf der Material Seite der eigenen Abteilung hinzugefügt. Buttons für den Excel Import und Export auf die Material Seite der eigenen Abteilung verschoben. Vorher unter Abteilung => Settings. Kleine visuelle Verbesserungen.
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.
Ich habe noch einen Bug: Ich kann 1x das Material importieren, aber wenn ich dann nochmals auf den Upload button klicke, kann ich das File auswählen aber danach passiert nichts. Das Modal mit der Tabellauswahl öffnet sich nicht. Auf der alten dev Instanz funktioniert das. Könntest du das bitte noch beheben?
Vielen Dank für dein refactoring und die neue Funktion! Finde ich super!
const { abteilung} = props; | ||
const [updateLoading] = useState(false); | ||
|
||
const delteMaterial = async () => { |
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.
kleiner Typo ;)
@@ -124,6 +127,18 @@ export const AbteilungMaterialView = (props: AbteilungMaterialViewProps) => { | |||
<AddStandortButton abteilungId={abteilung.id} /> | |||
</Can> | |||
</Col> | |||
<Can I={'delete'} this={{ __caslSubjectType__: 'Material', abteilungId: abteilung.id } as AbteilungEntityCasl}> |
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.
Ich bin nicht wirklich ein Fan von den Buttons unter der Navigation. Gibt es da einen besseren Weg?
Habt ihr dazu eine Meinung @jan-preisig @JeremiasBachmann @pemko123 ?
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.
UM alles Material zu löschen denke ich sollte dies beim Excel Import sein oder nicht?
Gehört irgendwie zusammen. Wann sonst sollte man alles MAterial löschen?
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.
Wie wäre es mit einem Dropdown in der Navigation namens Aktionen von wo aus man verschiedene Aktionen ausführen kann? Entweder direkt aus dem Dropdwon das popup öffnen oder eine Subpage mit funktionen?
Oder Option 2 wäre man nimmt alles zu den Einstellungen und hat dort einerseits funkionen um diverse dinge zu Bearbeiten und Erstellen sowie die Einstellungen.
Findet ihr davon etwas gut? @JeremiasBachmann @Leo1212 @pemko123
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.
Aktionen in der Navigation könnte man machen, solange es nicht zu viele werden. Unter Settings fände ich die Buttons irgendwie nicht sehr intuitiv, da Ich Buttons welche mit dem Material zu tun haben unter dem Menupunkt Material erwarten würde und nicht bei Settings. Ich finde die Buttons aber eigentlich nicht schlecht platziert, vor allem da sie ohnehin nur für das MatTeam/Admins sichtbar sind. Eine weiter Möglichkeit wäre das man für das MatTeam einen eigenen Menupunkt erstellt, in welchem sie das Mat verwalten können, und MatTeam spezifische Einstellung wie z.B. das aktivieren von Email Benachrichtigungen etc. haben.
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.
UM alles Material zu löschen denke ich sollte dies beim Excel Import sein oder nicht? Gehört irgendwie zusammen. Wann sonst sollte man alles MAterial löschen?
Beim Excel Import wurde dies so umgesetzt, dass man wählen kann ob das neue Material, das alte Material ersetzten soll oder ob es zum alten Material hinzugefügt werden soll.
> | ||
Abbrechen | ||
</Button>, | ||
<Tooltip title="Bestehendes Material wird mit dem Inhalt der Excel-Tabelle ergänzt."> |
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.
Ich habe da noch ein Tooltip hinzugefügt, damit man genau weiss was passiert.
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.
Perfekt, Dankeschön.
okText="Ja" | ||
cancelText="Nein" | ||
> | ||
<Tooltip title="Bestehendes Material wird gelöscht und mit dem Inhalt der Excel-Tabelle ersetzt."> |
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.
dito
@JeremiasBachmann Ich kann gerne das Approven. Das mit den Buttons ist okay für mich. Aber bitte noch den Bug beheben ;) |
Hab den Bug jetzt noch gefixed |
Funktioniert. Vielen Dank @jan-preisig |
Button um alles Material der Abteilung zu löschen auf der Material Seite der eigenen Abteilung hinzugefügt.
Änderung beim Importieren des Excels, muss nun ausgewählt werden ob das neue Material zum bestehenden Material der Abteilung hinzugefügt werden soll, oder ob das Bisherige Material ersetzt werden soll.
Buttons für den Excel Import und Export auf die Material Seite der eigenen Abteilung verschoben. Vorher unter Abteilung => Settings.
Kleine visuelle Verbesserungen.