Details des Tickets
Beschreibung,Kommentare und Anhänge
added 1 commit
- fcd5bc3c - altered the text
Compare with previous version |
Sascha Immig |
2018-12-17T09:56:33.891Z |
| added 1 commit
Compare with previous version |
Gregor Gabriel |
2018-12-18T08:05:38.394Z |
| Hier ist es besser eine interne VueJS Methode zu verwenden, statt einem jQuery events. |
Marcel Pociot |
2018-12-18T11:18:54.681Z |
| Das Flag kann als `prop` mit an die Komponente übergeben werden |
Marcel Pociot |
2018-12-18T11:19:17.161Z |
| Der Block sollte innerhalb der Komponente als eigene Methode genutzt werden. So muss nicht inline Javascript geladen werden. |
Marcel Pociot |
2018-12-18T11:19:53.911Z |
| Anstatt das über eine Klassen property zu lösen, kann man den Wert auch aus dem `$request` Objekt holen. So spart man sich die eigene Route und hat im Controller keine bool flags. |
Marcel Pociot |
2018-12-18T11:21:34.182Z |
added 22 commits
- 0e0e33ad...2a47410c - 21 commits from branch
develop - 8956efe0 - Merge branch 'develop' of http://gitlab.learningsystem.de/lms/seventeen into…
Compare with previous version |
Gregor Gabriel |
2018-12-19T08:45:01.169Z |
| klar hast du hier Recht,
wie sollte ich hier vorgehen?:
* Der Event-Consumer ist eh direkt unterhalb der UserAddressForm-Componente im selben Skript
* Der Code sollte dann in die Componente reingezogen werden
* Der reingezogene Code ist aber pure JS und sollte idealer weise nicht im Template-Teil sondern im JS-Teil der Vue-Componente residieren. Er enthält aber viele übersetzte Texte - **wie ziehe ich das sinnvoll in die vue-Componente**, |
Gregor Gabriel |
2018-12-19T09:13:04.820Z |
| Verstehe ich dich richtig, dass du aus dem Dialog erst mal eine Vue-Komponente machen würdest? Denn mir scheint der Block in keiner Komponente enthalten zu sein? |
Gregor Gabriel |
2018-12-19T09:16:52.600Z |
| Da verlasse ich mich auf deine Erfahrung.
Ich hatte hier eine eigene Route gebaut, weil ich dachte, dass der User damit auch anhand der URL erkennt, dass er auf einer speziellen Seite gelandet ist, die ihm das Fehlen von Daten anzeigt. Wenn das eher unüblich ist, dann nehme ich die neue Route gerne raus und verwende lediglich einen Request-Parameter. |
Gregor Gabriel |
2018-12-19T09:23:14.490Z |
| An welche Komponente denkst du konkret? |
Gregor Gabriel |
2018-12-19T09:23:46.103Z |
| changed this line in version 5 of the diff |
Gregor Gabriel |
2018-12-19T09:36:57.656Z |
| added 1 commit
Compare with previous version |
Gregor Gabriel |
2018-12-19T09:36:58.228Z |
| Habe das jetzt umgesetzt und submitted |
Gregor Gabriel |
2018-12-19T09:37:18.577Z |
added 5 commits
- 799b7a26...0e4c7917 - 4 commits from branch
develop - a4a3b4f3 - Merge branch 'develop' of http://gitlab.learningsystem.de/lms/seventeen into…
Compare with previous version |
Gregor Gabriel |
2018-12-19T09:39:03.832Z |
added 3 commits
- a4a3b4f3...a4d22c04 - 2 commits from branch
develop - c8c1505a - Merge branch 'develop' of http://gitlab.learningsystem.de/lms/seventeen into…
Compare with previous version |
Gregor Gabriel |
2018-12-19T09:55:53.248Z |
| Schau mal in der DiscussionItem.vue hier und hier. Dort werden bereits die PHP Übersetzungen in Javascript genutzt.
Um die Übersetzungsdatei zu aktualisieren kannst du den Artisan Befehl nutzen:
```
php artisan lang:js --no-lib
```
Anstatt des event consumers würde ich sowas machen wie:
```js
if(response.primary && this.updateProfile){
this.confirmProfileUpdate();
}
```
`updateProfile` wäre dann eine neue Property an der Komponente, die den bool Wert der aktuellen Blade IF-Abfrage enthält. |
Marcel Pociot |
2018-12-20T07:38:10.806Z |
| Ah du hast Recht. In dem Fall kann es sogar als jQuery im Template bleiben, da der Dialog sofort im document-ready event angezeigt wird. Wenn wir daraus erst eine Vue Komponente machen würden, hätten wir nicht wirklich was gewonnen. |
Marcel Pociot |
2018-12-20T07:40:54.318Z |
added 1 commit
- fbe1e98c - corrected typo
Compare with previous version |
Sascha Immig |
2018-12-20T13:14:05.824Z |
| merged |
Sascha Immig |
2018-12-20T13:36:06.493Z |
| Hallo Marcel,
habe Deine Vorschläge implementiert, bekomme aber keine Antwort, wenn ich die allererste Rechnungsadresse speichern möchte:
ReferenceError: response is not defined
Wo könnte das Problem liegen? Danke
section_invoice_addresses.blade.php
UserAddressForm.vue |
Zoltan Schnepf |
2019-01-04T13:15:29.414Z |
| Problem gelöst; Irreführung durch genesteten Funktionsruf: wegen Fehler in der Sub-Funktion im Catch-Block gelandet (natürlich dann ohne response).
Anmerkung: original feature 'invoice-adress-name-dialog' wurde gemerged & geschlossen => werde einen neuen Zweig aufmachen: 'invoice-address-name-dialog-discuss' (diesmal richtig mit doppel d) und die Korrektur dort veröffentlichen. |
Zoltan Schnepf |
2019-01-08T11:21:21.660Z |
| Title |
Id |
Author |
Created at |
Message |
| WIP: |
0e0e33ad48f760e70ed996359c4c79d3ec3dad38 |
TUV\gabrielg |
2018-12-18T08:05:30.000Z |
WIP:
+ updating email address to become users name, when accepting name from invoice address as user name
|
| Merge branch 'develop' of http://gitlab.learningsystem.de/lms/seventeen into… |
2b8b5f0d3bdb02435ea468507148e2b92b17cbfe |
TUV\gabrielg |
2018-12-14T07:44:18.000Z |
Merge branch 'develop' of http://gitlab.learningsystem.de/lms/seventeen into feature/invoice-adress-name-dialog
# Conflicts:
# public/js/admin/app.js
# public/js/admin/app.js.map
# public/js/app.js.map
|
| WIP: |
401333c64d632809582b09276602675d2de5969f |
TUV\gabrielg |
2018-12-12T16:16:35.000Z |
WIP:
added first and last name and gender to adress form for first address if user has already a name
|
| Merge branch 'develop' of http://gitlab.learningsystem.de/lms/seventeen into… |
68d5abf0bf0f02eb7608603cf5355ca7f47d010c |
TUV\gabrielg |
2018-12-13T12:01:40.000Z |
Merge branch 'develop' of http://gitlab.learningsystem.de/lms/seventeen into feature/invoice-adress-name-dialog
# Conflicts:
# public/css/app.css.map
# public/js/admin/app.js
# public/js/admin/app.js.map
# public/js/app.js
# public/js/app.js.map
|
| WIP: |
799b7a26f11ab9d9f67fd566d4b7c64feee150ab |
TUV\gabrielg |
2018-12-19T09:36:43.000Z |
WIP:
- removed separate route and used a request parameter to trigger missing name dialog in profile
|
| Merge branch 'develop' of http://gitlab.learningsystem.de/lms/seventeen into… |
8956efe0132bc9bcecd40b2fccff944a064c80f6 |
TUV\gabrielg |
2018-12-19T08:44:46.000Z |
Merge branch 'develop' of http://gitlab.learningsystem.de/lms/seventeen into feature/invoice-adress-name-dialog
# Conflicts:
# public/css/admin/style.css.map
# public/css/app.css.map
# public/js/admin/app.js
# public/js/admin/app.js.map
# public/js/app.js.map
|
| WIP: doing a booking while not having edited profile name ... leads to a dialog… |
898c9c60e01455c2206deef64d66ca704e46f988 |
TUV\gabrielg |
2018-12-13T10:28:37.000Z |
WIP: doing a booking while not having edited profile name ... leads to a dialog that suggests to take name ... from invoice adress
|
| Merge branch 'develop' of http://gitlab.learningsystem.de/lms/seventeen into… |
a4a3b4f381259946c8c6a1b1c39da35da40e9397 |
TUV\gabrielg |
2018-12-19T09:38:59.000Z |
Merge branch 'develop' of http://gitlab.learningsystem.de/lms/seventeen into feature/invoice-adress-name-dialog
|
| WIP: |
aa8dab1a383e7f36b2a583fbc22b7df8a9841802 |
TUV\gabrielg |
2018-12-14T08:45:17.000Z |
WIP:
+ accepting adress data as user data in profile now updates user form in profile
|
| WIP: after creating first address in profile an event is triggered about that.… |
c106398a8bb639a2d04b4880d64d1aa5160332f6 |
TUV\gabrielg |
2018-12-13T13:18:24.000Z |
WIP: after creating first address in profile an event is triggered about that. An event listener is installed, when user profile data is not complete
|
| Merge branch 'develop' of http://gitlab.learningsystem.de/lms/seventeen into… |
c8c1505ad12f1ec47070b1d741927a22a7aad6f1 |
TUV\gabrielg |
2018-12-19T09:55:49.000Z |
Merge branch 'develop' of http://gitlab.learningsystem.de/lms/seventeen into feature/invoice-adress-name-dialog
|
| Merge branch 'develop' of http://gitlab.learningsystem.de/lms/seventeen into… |
db6df281165d4faa2a689804158df91fb5d651b4 |
TUV\gabrielg |
2018-12-13T12:02:48.000Z |
Merge branch 'develop' of http://gitlab.learningsystem.de/lms/seventeen into feature/invoice-adress-name-dialog
|
| WIP: |
f1ae19b1dbf440d6cca824d4af7f055acf10d19c |
TUV\gabrielg |
2018-12-13T15:33:50.000Z |
WIP:
+ Generation of certificate disabled, if firstname, lastname or salutation is missing.
+ User is redirected to profile and gets an alert that data is missing for genersting certificate
+ If user accepts generatin, missing fieds are highlighned with error messages
|
| corrected typo |
fbe1e98c7d5613b3196a576e43a71894a4bc8bb5 |
TUV\immigs |
2018-12-20T13:13:53.000Z |
corrected typo
|
| altered the text |
fcd5bc3c2c3b76de0bd6a8e1ad50e9d5665d5422 |
TUV\immigs |
2018-12-17T09:56:20.000Z |
altered the text
|