-
Notifications
You must be signed in to change notification settings - Fork 57
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
20240205 feat floating flash messages #429
base: master
Are you sure you want to change the base?
Conversation
TODOs: - CSS für Design40 und Material anpassen - Fehler von t/flash_migration/deprecated_calls beheben
Erzeugt mit: find . -type f -exec sed -i '/^..-\? INCLUDE .common.flash.html. ..$/d' {} \;
SL::YAML benutzt YAML::XS wenn es installisert ist YAML::XS hat im Gegensatz zu YAML kein Stringify
Hallo, ja ich schaue es mir an. |
Falls möglich keine Pull Requests erstellen, bei denen die Testfälle nicht durchlaufen |
Hmm, Also ich wollte das jetzt mal testen und habe dazu den branch bei mir lokal ausgecheckt. Ich wollte dann ein Angebot erstellen, so dass ich allenfalls eine Flash Meldung erhalte. Da ist aber etwas kaputt. Wenn ich versuche einen Artikel hinzuzufügen unter "Artikel hier hinzufügen!", dann wird der Artikel nicht richtig übernommen und ich kann diesen auch nicht hinzufügen. Im Server log gibt es ein Wenn ich auf den master Branch switche geht es. So kann ich da nicht wirklich viel machen... Edit: Sehe da im log noch: Verstehe aber ehrlich gesagt nicht ganz woher das kommt... Wie kommt denn das jQuery in die Rose lib. |
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.
Ist hier Möglicherweise zuviel raus gefallen, weil das alles auf einer Zeile steht? Denke es bräuchte das T8.
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.
Hier auch? T8, L und LxERP.
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.
Hier auch. L, T8. Die Seite scheint zwar trotzdem zu funktionieren. Nein stimmt nicht die Seite ist kaputt. Ich war im neuen Design und das ist ja das alte.
Habe jetzt eine Test Seite erstellt mittels:
Es kommt keine Fehlermeldung aber auch keine Flash Meldung. |
Die CSS Klassen für das neue Design fehlt noch.
@rebootl Könntest du dir das mal anschauen?