Changed 'sie' to 'du' where appropriate #550
Conversation
choffmann
left a comment
choffmann
left a comment
There was a problem hiding this comment.
Great, thank you. I have made a few small suggestions. Could you also please rename the commit message to match our coding convention? See https://www.conventionalcommits.org/en/v1.0.0/
| Baum {loadedData.number} bearbeiten | ||
| </h1> | ||
| <p className="mb-5">In dieser Ansicht können Sie einem Baum bearbeiten.</p> | ||
| <p className="mb-5">In dieser Ansicht kannst du einem Baum bearbeiten.</p> |
There was a problem hiding this comment.
| <p className="mb-5">In dieser Ansicht kannst du einem Baum bearbeiten.</p> | |
| <p className="mb-5">In dieser Ansicht kannst du einen Baum bearbeiten.</p> |
| In dieser Ansicht kannst du eine neue Bewässerungsgruppe erstellen und dieser Bäume | ||
| zuweisen. |
There was a problem hiding this comment.
It's probably not your fault, but I think this sentence is difficult to read. Could you improve it?
| <div className={`text-red font-semibold text-sm mt-10 ${show ? '' : 'hidden'}`}> | ||
| <p className="mb-2"> | ||
| Es ist leider ein Problem aufgetreten. Bitte probieren Sie es erneut oder wenden Sie sich an | ||
| Es ist leider ein Problem aufgetreten. Bitte probiere es erneut oder wenden dich an |
There was a problem hiding this comment.
I think the term "versuchen" sounds better than "probieren"
| Es ist leider ein Problem aufgetreten. Bitte probiere es erneut oder wenden dich an | |
| Es ist leider ein Problem aufgetreten. Bitte versuche es erneut oder wenden dich an |
|
It would also be great if you could add an entry to the CHANGELOG.md 👍 |
LPaulikat
force-pushed
the
feature/siezen-to-duzen
branch
from
625c81f to
13eb91c
Compare
choffmann
left a comment
choffmann
left a comment
There was a problem hiding this comment.
Just one little thing that needs changing. The commit message is also not quite right; it should be refactor: changed "sie" to "du" where appropriate (with a space and lowercase). Sorry for being so picky, but it's for a nice commit history
Also remember to adjust the CHANGELOG.md file 😉
| In dieser Ansicht können Sie eine neue Bewässerungsgruppe erstellen sowie dieser Bäume | ||
| zuweisen. | ||
| </p> | ||
| In dieser Ansicht kannst du eine neue Bewässerungsgruppe erstellen und anschließend Bäume auswählen, die zu dieser Gruppe gehören sollen. </p> |
There was a problem hiding this comment.
Could you please remove the whitespace at the end of this text
choffmann
force-pushed
the
develop
branch
6 times, most recently
from
09ce218 to
d4dbf41
Compare
3 participants
Close green-ecolution/green-ecolution#64