entrouvert
/
chrono
14
1
Fork 0
Code Issues Pull Requests 4 Releases Wiki Activity

manager: add __str__ to MeetingType, translating deletion popup (#85718) #205

Merged
yweber merged 1 commits from wip/85718-meetingtype-delete-popup-translation into main 2024-01-17 13:28:51 +01:00
Conversation 1 Commits 1 Files Changed 2 +13
yweber commented 2024-01-17 11:18:19 +01:00
Owner
Copy Link
No description provided.
yweber added 1 commit 2024-01-17 11:18:19 +01:00
gitea/chrono/pipeline/head There was a failure building this commit Details
d6877a7e12 manager: add __str__ to MeetingType, translating deletion popup (#85718)
yweber force-pushed wip/85718-meetingtype-delete-popup-translation from d6877a7e12 to f349e48678 2024-01-17 11:20:08 +01:00 Compare
yweber force-pushed wip/85718-meetingtype-delete-popup-translation from f349e48678 to 47e7558298 2024-01-17 11:23:25 +01:00 Compare
yweber changed title from WIP: manager: add __str__ to MeetingType, translating deletion popup (#85718) to manager: add __str__ to MeetingType, translating deletion popup (#85718) 2024-01-17 11:29:54 +01:00
vdeniaud approved these changes 2024-01-17 12:26:33 +01:00
vdeniaud left a comment
Owner
Copy Link

Puisque je ne trouve rien à redire au patch je me permets une remarque sur le message de commit que je trouve un peu verbeux :) j'aurais écrit : manager: translate title on meeting type deletion page (#85718).

(dans l'idée notamment d'éviter paraphraser le code, ce qui est pour moi rarement utile (la partie add __str__ to MeetingType))

Puisque je ne trouve rien à redire au patch je me permets une remarque sur le message de commit que je trouve un peu verbeux :) j'aurais écrit : `manager: translate title on meeting type deletion page (#85718)`. (dans l'idée notamment d'éviter paraphraser le code, ce qui est pour moi rarement utile (la partie `add __str__ to MeetingType`))
yweber commented 2024-01-17 13:28:28 +01:00
Author
Owner
Copy Link

Puisque je ne trouve rien à redire au patch je me permets une remarque sur le message de commit que je trouve un peu verbeux :) j'aurais écrit : manager: translate title on meeting type deletion page (#85718).

(dans l'idée notamment d'éviter paraphraser le code, ce qui est pour moi rarement utile (la partie add __str__ to MeetingType))

Merci ! C'est noté.

> Puisque je ne trouve rien à redire au patch je me permets une remarque sur le message de commit que je trouve un peu verbeux :) j'aurais écrit : `manager: translate title on meeting type deletion page (#85718)`. > > (dans l'idée notamment d'éviter paraphraser le code, ce qui est pour moi rarement utile (la partie `add __str__ to MeetingType`)) > > Merci ! C'est noté.
yweber merged commit 47e7558298 into main 2024-01-17 13:28:51 +01:00
yweber deleted branch wip/85718-meetingtype-delete-popup-translation 2024-01-17 13:28:51 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
vdeniaud
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
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: entrouvert/chrono#205
No description provided.
Delete Branch "wip/85718-meetingtype-delete-popup-translation"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?