Page MenuHomePhorge

Remove useless items and added missing items of create-room-page series
ClosedPublic

Authored by nannanko on Sat, Oct 26, 6:49 PM.
Tags
None
Referenced Files
F112191: D194.1732275227.diff
Thu, Nov 21, 3:33 AM
F112190: D194.1732275120.diff
Thu, Nov 21, 3:32 AM
F112053: D194.1732258324.diff
Wed, Nov 20, 10:52 PM
Subscribers
None

Details

Summary

create-room-page-visibility is not used anywhere, this revision removed them.
cmn-Hans miss create-room-page-type and create-room-invite items, this revision added them.

Type: add

Test Plan

N/A

Diff Detail

Repository
rK kazv
Branch
nannanko/stacked
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 309
Build 606: GitLab CI for kazv
Build 605: arc lint + arc unit

Event Timeline

Please add a changelog type to the summary, using the following format:

Type: (add|remove|skip|security|fix)

  • Keep room types consistent with old translations
Harbormaster returned this revision to the author for changes because remote builds failed.Sat, Oct 26, 7:03 PM
Harbormaster failed remote builds in B307: Diff 513!
  • Improve translation of create-room-page-type-direct
  • Fix translations inconsistent
tusooa requested changes to this revision.Fri, Nov 1, 7:22 PM
] wc -l src/l10n/cmn-Hans/100-ui.ftl
413 src/l10n/cmn-Hans/100-ui.ftl
] wc -l src/l10n/en/100-ui.ftl   
435 src/l10n/en/100-ui.ftl

seems there are still a lot of things missing in mandarin

This revision now requires changes to proceed.Fri, Nov 1, 7:22 PM
In D194#3576, @tusooa wrote:
] wc -l src/l10n/cmn-Hans/100-ui.ftl
413 src/l10n/cmn-Hans/100-ui.ftl
] wc -l src/l10n/en/100-ui.ftl   
435 src/l10n/en/100-ui.ftl

seems there are still a lot of things missing in mandarin

No, there is only one name error rather than lack of translation items.

$ grep '=' src/l10n/cmn-Hans/100-ui.ftl | awk '{print $1}' > ~/.tmp/cmn
$ grep '=' src/l10n/en/100-ui.ftl | awk '{print $1}' > ~/.tmp/en
$ diff ~/.tmp/{cmn,en}
308c308
< user-page-ban-user-confirm-dialog-titile
---
> user-page-ban-user-confirm-dialog-title

Maybe we need a CI script to check the consistency of translation items.

The collide of line counts is due to the different formats for some same items.

e.g.

// cmn-Hans
room-list-view-room-item-unread-notification-count-text = { $count } 条未读通知

// en
room-list-view-room-item-unread-notification-count-text = { $count } unread {
    $count ->
        [1] message
        *[other] messages
    }

Ok, I will work on such script.

This revision is now accepted and ready to land.Sat, Nov 2, 7:37 AM