Netz39_Vorstand/prozesse

Fork 0

Remove json stuff to show mail text correctly #22

Merged

MG-95 merged 1 commit from mailtext into master

2022-12-04 22:21:09 +01:00

MG-95 commented

2022-12-04 22:06:21 +01:00

Contributor

No description provided.

MG-95 added 1 commit

2022-12-04 22:06:23 +01:00

Remove json stuff to show mail text correctly b9c8ee617b

requested reviews from tux, dkdent

2022-12-04 22:06:29 +01:00

tux commented

2022-12-04 22:10:15 +01:00

Owner

Was spricht dagegen, beides in der Description zu haben?

dkdent approved these changes

2022-12-04 22:10:22 +01:00

MG-95 commented

2022-12-04 22:12:54 +01:00

Author

Contributor

Was spricht dagegen, beides in der Description zu haben?

aktuell sieht es so aus und das ist eher hinderlich

> Was spricht dagegen, beides in der Description zu haben? aktuell sieht es so aus und das ist eher hinderlich ![grafik](/attachments/8faad224-5689-4493-9945-35843b887bb8)

grafik.png

40 KiB

dkdent commented

2022-12-04 22:13:02 +01:00

Collaborator

Was spricht dagegen, beides in der Description zu haben?

die Idee war ja ursprünglich, dass wir die komplette description nehmen, um die Mail automatisch zu verschicken. Das ist in den letzten 4 Jahren nicht passiert ;-)

Wir kriegen da mit sehr geringem Aufwand auch wieder das json in die Description. Jetzt erstmal ist alles besser damit geholfen, wenn da nur der zu kopierende Text drin steht, denke ich.

> Was spricht dagegen, beides in der Description zu haben? die Idee war ja ursprünglich, dass wir die komplette description nehmen, um die Mail automatisch zu verschicken. Das ist in den letzten 4 Jahren nicht passiert ;-) Wir kriegen da mit sehr geringem Aufwand auch wieder das json in die Description. Jetzt erstmal ist alles besser damit geholfen, wenn da nur der zu kopierende Text drin steht, denke ich.

👍 1

MG-95 commented

2022-12-04 22:14:19 +01:00

Author

Contributor

Zumindest weiß ich nicht, wie man mit Thunderbird JSON-Zeug einliest

MG-95 commented

2022-12-04 22:15:10 +01:00

Author

Contributor

Was spricht dagegen, beides in der Description zu haben?

die Idee war ja ursprünglich, dass wir die komplette description nehmen, um die Mail automatisch zu verschicken. Das ist in den letzten 4 Jahren nicht passiert ;-)

Wir kriegen da mit sehr geringem Aufwand auch wieder das json in die Description. Jetzt erstmal ist alles besser damit geholfen, wenn da nur der zu kopierende Text drin steht, denke ich.

ah okay, ja das macht Sinn.
Wenn die Automatismus da ist, kann man die Commits ja reverten :)

> > Was spricht dagegen, beides in der Description zu haben? > > die Idee war ja ursprünglich, dass wir die komplette description nehmen, um die Mail automatisch zu verschicken. Das ist in den letzten 4 Jahren nicht passiert ;-) > > Wir kriegen da mit sehr geringem Aufwand auch wieder das json in die Description. Jetzt erstmal ist alles besser damit geholfen, wenn da nur der zu kopierende Text drin steht, denke ich. ah okay, ja das macht Sinn. Wenn die Automatismus da ist, kann man die Commits ja reverten :)

tux commented

2022-12-04 22:17:13 +01:00

Owner

Was spricht dagegen, beides in der Description zu haben?

aktuell sieht es so aus und das ist eher hinderlich

Spannend - im Quellcode steht was anderes. 8)

> > Was spricht dagegen, beides in der Description zu haben? > > aktuell sieht es so aus und das ist eher hinderlich > > ![grafik](/attachments/8faad224-5689-4493-9945-35843b887bb8) Spannend - im Quellcode steht was anderes. 8)

tux approved these changes

2022-12-04 22:17:22 +01:00

MG-95 merged commit 5f8688b13e into master

2022-12-04 22:21:09 +01:00

MG-95 deleted branch mailtext

2022-12-04 22:21:11 +01:00

MG-95 referenced this pull request from a commit

2022-12-04 22:21:11 +01:00

Merge pull request 'Remove json stuff to show mail text correctly' (#22) from mailtext into master