Move description from title to paragraph #8

Merged
j merged 0 commits from better-eventview into master 2020-07-25 22:16:44 +00:00
j commented 2020-07-25 19:59:49 +00:00 (Migrated from git.zeus.gent)

The description of events can be quite long: they contain a description
of what will happen during the event and what safety measures are taken.

The description of events can be quite long: they contain a description of what will happen during the event and what safety measures are taken.
midgard commented 2020-07-25 20:01:14 +00:00 (Migrated from git.zeus.gent)

/approve

/approve
flynn commented 2020-07-25 20:02:39 +00:00 (Migrated from git.zeus.gent)

kan ook, ik dacht dat een event dan mss te groot ging worden en dat we vooral korte beschrijvingen gingen hebben dus had het daarom bijgeplaatst op de eerste lijn.

ook omdat de note al snel vrij belangrijk kan zijn.

Misschien hem wel vet laten dan?

kan ook, ik dacht dat een event dan mss te groot ging worden en dat we vooral korte beschrijvingen gingen hebben dus had het daarom bijgeplaatst op de eerste lijn. ook omdat de note al snel vrij belangrijk kan zijn. Misschien hem wel vet laten dan?
midgard commented 2020-07-25 20:07:09 +00:00 (Migrated from git.zeus.gent)

Ik zou zo'n hele alinea niet vet zetten.

Ik zou zo'n hele alinea niet vet zetten.
flynn commented 2020-07-25 20:07:39 +00:00 (Migrated from git.zeus.gent)

Of parsen en de eerste zin vet zetten? (mss wat veel werk)

Of parsen en de eerste zin vet zetten? (mss wat veel werk)
midgard commented 2020-07-25 20:08:04 +00:00 (Migrated from git.zeus.gent)

Eerste lijn als titel zoals bij Git is een mogelijkheid

Eerste lijn als titel zoals bij Git is een mogelijkheid
j commented 2020-07-25 20:09:33 +00:00 (Migrated from git.zeus.gent)

Of markdown in description?

Of markdown in description?
midgard commented 2020-07-25 20:09:58 +00:00 (Migrated from git.zeus.gent)
split = description.split("\n\n", maxsplit=1)
title, body = split if len(split) == 2 else (split[0], body)
```python split = description.split("\n\n", maxsplit=1) title, body = split if len(split) == 2 else (split[0], body) ```
j commented 2020-07-25 20:14:02 +00:00 (Migrated from git.zeus.gent)

Dat code-snippet werkt toch niet?

Dat code-snippet werkt toch niet?
midgard commented 2020-07-25 20:18:58 +00:00 (Migrated from git.zeus.gent)
>>> description="Hallo wereld\n\nDit is een event"
>>> split = description.split("\n\n", maxsplit=1)
>>> title, body = split if len(split) == 2 else (split[0], body)
>>> title
'Hallo wereld'
>>> body
'Dit is een event'
``` >>> description="Hallo wereld\n\nDit is een event" >>> split = description.split("\n\n", maxsplit=1) >>> title, body = split if len(split) == 2 else (split[0], body) >>> title 'Hallo wereld' >>> body 'Dit is een event' ```
j commented 2020-07-25 20:22:34 +00:00 (Migrated from git.zeus.gent)

akkoord, maar voor de case waarbij er geen 2 newlines staan tho?

akkoord, maar voor de case waarbij er geen 2 newlines staan tho?
midgard commented 2020-07-25 20:26:36 +00:00 (Migrated from git.zeus.gent)

Als je dat wil ondersteunen, schrijf je gewoon "\n" ipv "\n\n"

Als je dat wil ondersteunen, schrijf je gewoon `"\n"` ipv `"\n\n"`
j commented 2020-07-25 20:27:32 +00:00 (Migrated from git.zeus.gent)

nee, ik bedoel dat je else niet werkt:

>>> description = "testing testing"
>>> split = description.split("\n\n", maxsplit=1)
>>> title, body = split if len(split) == 2 else (split[0], body)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
NameError: name 'body' is not defined
nee, ik bedoel dat je else niet werkt: ``` >>> description = "testing testing" >>> split = description.split("\n\n", maxsplit=1) >>> title, body = split if len(split) == 2 else (split[0], body) Traceback (most recent call last): File "<stdin>", line 1, in <module> NameError: name 'body' is not defined ```
midgard commented 2020-07-25 21:05:57 +00:00 (Migrated from git.zeus.gent)

Ah, juist

Ah, juist
j commented 2020-07-25 22:16:38 +00:00 (Migrated from git.zeus.gent)

Ik ga het voorlopig mergen, als mensen het niet duidelijk vinden, gaan ze dat wel zeggen en een MR maken

Ik ga het voorlopig mergen, als mensen het niet duidelijk vinden, gaan ze dat wel zeggen en een MR maken
j commented 2020-07-25 22:16:44 +00:00 (Migrated from git.zeus.gent)

merged

merged
j commented 2020-07-25 22:16:45 +00:00 (Migrated from git.zeus.gent)

mentioned in commit d107409619

mentioned in commit d1074096193f761e96cf468cfe6ddf7010561a29
midgard commented 2020-07-25 22:47:53 +00:00 (Migrated from git.zeus.gent)

gaan ze [wel] een MR maken

Lol.

> gaan ze [wel] een MR maken Lol.
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
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: ZeusWPI/kers#8
No description provided.