From d1f5ee6bf1e4c8b6b8d16cd0f316a5c076e2879d Mon Sep 17 00:00:00 2001 From: nhienlam Date: Thu, 9 Nov 2023 09:56:05 -0800 Subject: [PATCH] Cleanup --- translations/en-GB.xtb | 18 ------------------ 1 file changed, 18 deletions(-) diff --git a/translations/en-GB.xtb b/translations/en-GB.xtb index 75762647..952e492d 100644 --- a/translations/en-GB.xtb +++ b/translations/en-GB.xtb @@ -46,24 +46,6 @@ Cook Islands Wrong code. Try again. Macau -pavithrar -Any idea why these strings got removed? -nhienlam -I just checked. These got removed because they were removed from the source code (soy msg tag). Some string got modified in source code and new translations were exported, so there were some duplicated (or slightly different) translations in these files. The old translations, whose strings were removed in source code, would be removed here. Eg. Removed from source code: -2070709910567585968 (removed in cl/386349750) -1929332760123475919 -1966996824429980990 Modified: -2016421613709283485 (new 4571870355114729513) -6295157125625453859 (new 2187260952722791093) -8971555482255213064 (new 1630237004390596779 and 4877597386645466815) -nhienlam -One concern: We will manually copy these translations to github. But if there were strings that were only changed in google3 and not github (eg. cl/386349750 vs https://github.com/firebase/firebaseui-web/blob/85fbdd46847dff45afe76d9341e8c001a3394e2d/soy/strings.soy#L244), we shouldn't remove the old translations when copying over the translations to github... Before copying the translations over to github, we need to update all the strings to be the same as google3 so that the translations work. -pavithrar -that's exactly my concern, cos firebase UI web contains these strings - https://github.com/firebase/firebaseui-web/blob/85fbdd46847dff45afe76d9341e8c001a3394e2d/soy/strings.soy#L244 We can either 1) manually change this CL and revert any deletions, then copy over translations as is to github. 2) Update strings.soy in firebase UI web to match google3 and then release a new UI Web version - I think this might be a more involved change. What about we manually fix up this CL to only add things and not remove any strings? Then we just need to publish translations + update gstatic.. no need of a ui-web release right? -nhienlam -To fix b/297769205, I'm fine with option 1. But in the long run, we should update the github strings to match google3 some time in the future so that we won't need to manually revert the deletions every time we run //third_party/firebaseui_web/tools:import_translations. What about we manually fix up this CL to only add things and not remove any strings? Then we just need to publish translations + update gstatic.. no need of a ui-web release right? Even if we do option 1, we still need to do ui-web release, don't we? If we deploy gstatic (cdn), we should also release the github ui-web, right? Otherwise, gstatic will have v6.1.1 while github doesn't have v6.1.1. -pavithrar -Oct 5, 3:26 PM -Synced up offline and decided that we will submit this CL, then release on gstatic as version 6.1.1. this will show up as "https://www.gstatic.com/firebasejs/ui/6.1.1/firebase-ui-auth__fr.js" for example.. But github users of firebaseui-web wont know to pick it up. - -Once we verify this gstatic version with web widgets, we will also copy over the translations to github, make password policy error changes in github and release a new uiweb version 6.1.1. This will be consistent with what is in gstatic. - -TLDR - i am ok with this CL getting submitted as is. -Resolved The password must be at least six characters long Sign in with Enter the 6-digit code that we sent to