Sloppy language #224
No reviewers
Labels
No labels
Component: User interface
Component: Wymeditor
Help wanted
Level: Difficult
Level: Easy
Level: Moderate
Pagetype: Festival
Pagetype: Mailinglist
Pagetype: Peer reviewed article
Pagetype: Text
Privacy GDPR AVG
status: has conflicts
Status: Needs changes
Status: Needs discussion
Status: Needs review
Status: Ready to merge
Status: Waiting for response
Type: Bug
Type: Enhancement
Type: Question
Usecase: De Stadsbron
Usecase: Koppelting
Usecase: MeetjeStad
Value: Coders
Value: Security
Value: Users
Value: Visitors
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
harmen/hypha!224
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "sloppy-language"
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?
Most of these changes look good. I added some inline comments. Also,
login-username,login-passwordandadd-mailingare introduced as strings in the first commit, but only used in the second commit. It would be better to group both adding and using these strings in a single commit (ideally in a separate commit from the other string changes, but putting everything in a single commit would also be ok for me, since it all improves or adds translations).I can't find any reference to this string in the code?
@ -15,3 +15,3 @@"logout" => "uitloggen","quit" => "stop","quit" => "stoppen","logged-in-as" => "ingelogd als",I think this conflicts with (and is more elaborately fixed by) #218, so probably best to remove this change from this commit.
Ah, seems this was added in #227 (both the string definition and the usage). I think this should be dropped from this commit.
Scratch that, it seems #227 only adds the strings, but does not use them anywhere...
Looks to me like
mlrefers to mailinglists. Can they have comments?Looks good at first glance. I think this can be merged after testing.
Tested, works. I rebased on top of master, merging now.